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: us...@infra.apache.org