jayzhan211 commented on code in PR #10268: URL: https://github.com/apache/datafusion/pull/10268#discussion_r1581974152
########## datafusion/expr/src/type_coercion/functions.rs: ########## @@ -450,19 +473,7 @@ fn coerced_from<'a>( { Some(type_into.clone()) } - // More coerce rules. - // Note that not all rules in `comparison_coercion` can be reused here. - // For example, all numeric types can be coerced into Utf8 for comparison, - // but not for function arguments. - _ => comparison_binary_numeric_coercion(type_into, type_from).and_then( Review Comment: This logic is introduced in #9459, so I think it is safe to remove together with this PR -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org