cloud-fan commented on pull request #28269: URL: https://github.com/apache/spark/pull/28269#issuecomment-655365714
I think so, but now we know that another advantage of `InSet` is to reduce analyze/optimize time. Maybe we should create a base class for `In` and `InSet`, so that these 2 classes are only different in the expression tree, eval and codegen are the same. ---------------------------------------------------------------- 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]
