You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "mustafasrepo (via GitHub)" <gi...@apache.org> on 2023/07/25 10:45:15 UTC

[GitHub] [arrow-datafusion] mustafasrepo commented on a diff in pull request #7083: Fix cli tests

mustafasrepo commented on code in PR #7083:
URL: https://github.com/apache/arrow-datafusion/pull/7083#discussion_r1273350819


##########
datafusion/core/tests/sqllogictests/test_files/aggregate.slt:
##########
@@ -95,9 +95,16 @@ SELECT approx_percentile_cont(c3, 0.95, c1) FROM aggregate_test_100
 statement error Error during planning: The percentile sample points count for ApproxPercentileCont must be integer, not Float64.
 SELECT approx_percentile_cont(c3, 0.95, 111.1) FROM aggregate_test_100
 
-# csv_query_array_agg_unsupported
-statement error This feature is not implemented: Order-sensitive aggregators is not supported on multiple partitions
-SELECT array_agg(c13 ORDER BY c1) FROM aggregate_test_100
+# array agg can use order by

Review Comment:
   To make result deterministic, I have changed the test



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