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 2021/06/14 08:08:37 UTC

[GitHub] [arrow-datafusion] houqp edited a comment on pull request #55: Support qualified columns in queries

houqp edited a comment on pull request #55:
URL: https://github.com/apache/arrow-datafusion/pull/55#issuecomment-860150632


   @andygrove @Dandandan @jorgecarleitao @alamb @Jimexist this PR is now ready for review. We are now able to pass both tpch-7 and tpch-8.
   
   I filed https://github.com/apache/arrow-datafusion/issues/549 to track compound column support as a fast follow task since this PR is already very large and keeps growing every time I rebase on top of the latest master. If you all think I should include compound column support in this PR, I am more than happy to add it.
   
   Output field name are not 100% conforming to https://github.com/apache/arrow-datafusion/blob/master/docs/specification/output-field-name-semantic.md yet. The remaining differences are all minor formatting issues like converting function names to lower cases, so I decided to leave that to follow up PRs to reduce the diff.


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

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