peter-toth commented on PR #40473:

   Hm, I think you are right @Kimahriman, `LambdaVariable` and 
`NamedLambdaVariable` are very different and `NamedLambdaVariable` seem to be 
used only in `LambdaFunction`s, so 
doesn't make sense and actually it can prevent pulling out higher order 
functions and so cause performance regression... I think that PR should be 
   But I feel that is orthogonal to the issue that we use 
`EquivalentExpressions` for different purposes in `PhysicalAggregation` (the 
only place where we use `.addExpr()`) and in executors (`.addExprTree()` for 
subexpression elimination).

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:

For queries about this service, please contact Infrastructure at:

To unsubscribe, e-mail:
For additional commands, e-mail:

Reply via email to