alex-plekhanov commented on code in PR #12148:
URL: https://github.com/apache/ignite/pull/12148#discussion_r2175428134


##########
modules/calcite/src/test/java/org/apache/ignite/internal/processors/query/calcite/integration/AggregatesIntegrationTest.java:
##########
@@ -116,6 +119,57 @@ public void testCountWithBackupsAndCacheModes() {
         assertQuery("select count(*) from person").returns(7L).check();
     }
 
+    /** TODO https://issues.apache.org/jira/browse/IGNITE-25765 : unignore 
after the fix. */
+    @Ignore
+    @Test
+    public void testArrayConcatAgg() {
+        sql("CREATE TABLE tarr(val INT, arrn INTEGER ARRAY, arrnn INTEGER 
ARRAY NOT NULL, arrn2 INTEGER ARRAY) WITH "
+            + atomicity());
+
+        sql("INSERT INTO tarr VALUES (1, null, ARRAY[1,2,3], ARRAY[10,11,12]), 
(2, ARRAY[4,5,6], ARRAY[7,8,9], null)");
+
+        assertQuery("SELECT ARRAY_CONCAT_AGG(a) from (select arrn from tarr 
union all select arrnn from tarr) T(a)")
+            .returns(IntStream.range(1, 
8).boxed().collect(Collectors.toList()))

Review Comment:
   `F.asList(1, 2, 3, 4, 5, 6, 7)` is shorter and easier to read



-- 
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: notifications-unsubscr...@ignite.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to