[GitHub] [incubator-pinot] jackjlli commented on a change in pull request #6192: Reduce number of sql queries from 10k to 500 in pinot-integration-tests module

[GitBox <gi...@apache.org>]

jackjlli commented on a change in pull request #6192:
URL: https://github.com/apache/incubator-pinot/pull/6192#discussion_r513819596



##########
File path: pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/ClusterIntegrationTestUtils.java
##########
@@ -859,7 +859,9 @@ static void testSqlQuery(String pinotQuery, String brokerUrl, org.apache.pinot.c
     BrokerRequest brokerRequest =
         PinotQueryParserFactory.get(CommonConstants.Broker.Request.SQL).compileToBrokerRequest(pinotQuery);
     if (isSelectionQuery(brokerRequest)) { // selection
-      // TODO: compare results for selection queries, w/o order by
+      // TODO: 1. compare results for selection queries, w/o order by.
+      //       2. validate values of multi-value columns.
+      //       3. remove the restriction that order by column has to be in selection clause.

Review comment:
       I can add the issue number in the commit.




----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org