Dandandan commented on pull request #1368:
URL: https://github.com/apache/arrow-datafusion/pull/1368#issuecomment-980539201
thanks @viirya for the quick change / PR! I think this is a good change.
I am wondering if there's more places where we don't want to add an alias.
Also we probably need some more tests to cover filter pushdown in
combination with other optimization passes like this one.
Another thing to consider might be for the filter operation to ignore the
alias.
FYI @alamb WDYT
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
