Dandandan commented on pull request #9376: URL: https://github.com/apache/arrow/pull/9376#issuecomment-774459834
I think changes here look good, and will be helpful to implement functions which typically operate on `(scalar, array)`. And this will help to make those cases (quite a bit) more efficient. I think that can be particularly helpful when having complex expressions in `WHERE` clauses / projections. @alamb I am not sure I understand why folding should be done on the physical plan level or really depending on this PR , it should be possible without changes in this PR (just by having some `Expr -> Expr` rules)? They could share the same evaluation code though? ---------------------------------------------------------------- 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]
