hassaanch23 commented on PR #25400:
URL: https://github.com/apache/datafusion/pull/25400#issuecomment-5994139842

   Thanks @jayzhan211! Both are added in 61ae8fb8b, right after the 
`array_agg`/`string_agg` query, with the expected results you gave. They check 
out against the table: the two groups' `k` values are {2, 1} and {4, 3}.
   
   Without this PR, both queries fail on `main` with `In an aggregate with 
DISTINCT, ORDER BY expressions must appear in argument list`; with it, they 
return those results. I also ran `aggregate.slt` with this branch merged into 
today's `main`.
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to