This is an automated email from the ASF dual-hosted git repository.

kfaraz pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git


The following commit(s) were added to refs/heads/master by this push:
     new ca0064ef47a test(processing): Fix flaky index cursor ordering in 
IncrementalIndexCursorFactoryTest (#19857)
ca0064ef47a is described below

commit ca0064ef47a5403205e71bad4a3d5371f554c52b
Author: Frank Chen <[email protected]>
AuthorDate: Mon Aug 3 12:31:31 2026 +0800

    test(processing): Fix flaky index cursor ordering in 
IncrementalIndexCursorFactoryTest (#19857)
---
 .../incremental/IncrementalIndexCursorFactoryTest.java      | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git 
a/processing/src/test/java/org/apache/druid/segment/incremental/IncrementalIndexCursorFactoryTest.java
 
b/processing/src/test/java/org/apache/druid/segment/incremental/IncrementalIndexCursorFactoryTest.java
index 9a447dd39e0..cfd89f73eaf 100644
--- 
a/processing/src/test/java/org/apache/druid/segment/incremental/IncrementalIndexCursorFactoryTest.java
+++ 
b/processing/src/test/java/org/apache/druid/segment/incremental/IncrementalIndexCursorFactoryTest.java
@@ -232,11 +232,14 @@ public class IncrementalIndexCursorFactoryTest extends 
InitializedNullHandlingTe
 
     Assert.assertEquals(2, results.size());
 
-    ResultRow row = results.get(0);
-    Assert.assertArrayEquals(new Object[]{null, "bo", 1L}, row.getArray());
-
-    row = results.get(1);
-    Assert.assertArrayEquals(new Object[]{"hi", null, 1L}, row.getArray());
+    // GroupingEngine.process returns raw grouped results without applying the 
query's order-by post-processing.
+    // The result order is therefore not guaranteed when the incremental index 
is not sorted by dimensions.
+    Assert.assertTrue(
+        results.stream().anyMatch(row -> Arrays.deepEquals(new Object[]{null, 
"bo", 1L}, row.getArray()))
+    );
+    Assert.assertTrue(
+        results.stream().anyMatch(row -> Arrays.deepEquals(new Object[]{"hi", 
null, 1L}, row.getArray()))
+    );
   }
 
   @Test


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

Reply via email to