jorgecarleitao edited a comment on pull request #7796: URL: https://github.com/apache/arrow/pull/7796#issuecomment-662800174
The test `csv_query_group_by_avg_with_projection` is now failing (after rebase). The query is `SELECT avg(c12), c1 FROM aggregate_test_100 GROUP BY c1` and the error is that the order of the result is wrong: it is `c1, avg(c12)`, not `c1, avg(c12)`. I am looking into it. The root cause is that `projection_expr` is not passed to the `LogicalPlanBuilder` from the sql planner, and thus there is no way for the `LogicalPlanBuilder` to know which order to apply when building the schema (it uses the order: groups first, aggregations second) ---------------------------------------------------------------- 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
