alamb commented on pull request #8654: URL: https://github.com/apache/arrow/pull/8654#issuecomment-727196735
> Tests need fixing with the new behavior (un-sorted aggregate results). I think that we could sort the testing results in native rust (as opposed to in datafusion), so that when a test fails, it is easier to figure out whether it was the sort node or the aggregate node that caused the issue. In past lives, we have addressed the "output order of group by may not be sorted" by normalization in the test suite itself (aka sort the output record batch in the *test* itself). ---------------------------------------------------------------- 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: [email protected]
