maropu edited a comment on pull request #28983: URL: https://github.com/apache/spark/pull/28983#issuecomment-654521636
> @cloud-fan @maropu One remaining issue: this rule works for array types but does not do casting, such as array(int) to array(double). IMO this isn't a deal breaker but if there is an easy way to implement that kind of casting, I can include it. In that case, wouldn't type mismatch exceptions be thrown in terms of type-safty? Anyway, I think its okay to focus on adding a new rule to resolve expessions in `ExpressionEcoder` here as @cloud-fan @viirya suggested above. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
