You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/10/06 10:45:40 UTC

[GitHub] [arrow-datafusion] alamb commented on pull request #3727: Consolidate and better tests for expression re-rewriting / aliasing

alamb commented on PR #3727:
URL: https://github.com/apache/arrow-datafusion/pull/3727#issuecomment-1269792375

   > I think there are may place which use the ExprRewriter to rewrite the expr in the optimizer like ConstEvaluator and other.
   >  Do we need to fix all of them?
   
   That is an excellent point @liukun4515  -- I think we probably do. I will file a follow on ticket to track and do so


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org