liukun4515 commented on PR #3727: URL: https://github.com/apache/arrow-datafusion/pull/3727#issuecomment-1269873727
> > 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 I have less knowledge about the alias for the expr, can you explain when and where we should do the alias for the expr which will be rewrited. -- 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]
