It might make sense to remove the catch all from this `match` expression https://github.com/apache/arrow-datafusion/blob/bc0ba6a724aaaf312b770451718cbce696de6640/datafusion/optimizer/src/analyzer/type_coercion.rs#L413 And explicitly list out all Expr types so that we don't accidentally another Expr type in the future Is that something you might be willing to do in another PR @guojidan ? _Originally posted by @alamb in https://github.com/apache/arrow-datafusion/pull/8982#discussion_r1464993523_
https://github.com/apache/arrow-datafusion/blob/bc0ba6a724aaaf312b770451718cbce696de6640/datafusion/optimizer/src/analyzer/type_coercion.rs#L413
And explicitly list out all Expr types so that we don't accidentally another Expr type in the future
Is that something you might be willing to do in another PR @guojidan ?
Originally posted by @alamb in #8982 (comment)