[GitHub] [arrow-datafusion] jackwener commented on pull request #4141: fix: shouldn't pass alias through into subquery.

[GitBox <gi...@apache.org>]

jackwener commented on PR #4141:
URL: https://github.com/apache/arrow-datafusion/pull/4141#issuecomment-1311637198

   @alamb @andygrove 
   I have find the reason of `projection include [c1, c2]`.
   It's because the `pushdown project` rule don't handle `alias`. In fact, when handle `projection with alias`, we need replace the `alias column` in `new_required_columns`.
   
   For example:
   
   ```
   project c1 as c2
       tablescan c1.
   ```
   
   when we meet `project c1 as c2`, if `new_required_columns` include c2, we need replace it with c1.


-- 
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