atris commented on a change in pull request #7141:
URL: https://github.com/apache/incubator-pinot/pull/7141#discussion_r666685741
##########
File path:
pinot-core/src/test/java/org/apache/pinot/queries/DistinctQueriesTest.java
##########
@@ -245,12 +245,14 @@ public void testSingleColumnDistinctOnlyInnerSegment()
// String columns
//@formatter:off
List<String> queries = Arrays
- .asList("SELECT DISTINCT(stringColumn) FROM testTable", "SELECT
DISTINCT(rawStringColumn) FROM testTable");
+ .asList("SELECT DISTINCT(stringColumn) FROM testTable");
Review comment:
We need the changes since the order of results in absence of ORDER BY is
not consistent between segment reads and dictionary reads.
--
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: commits-unsubscr...@pinot.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
