[GitHub] [arrow] jorisvandenbossche commented on issue #34501: [C++] Substrait plan with multiple aggregate fields returns incorrect results

["jorisvandenbossche (via GitHub)" <gi...@apache.org>]

jorisvandenbossche commented on issue #34501:
URL: https://github.com/apache/arrow/issues/34501#issuecomment-1460374432

   I can't directly find the issue or PR, but I remember we recently discussed with @westonpace about having the group key column first vs last. Acero puts it last, but we probably want to change that to first (matching what R, pandas, .. and others do). 
   It might be that substrait is also expecting that? (the output field names in the plan are in that order)


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