somandal commented on code in PR #9810:
URL: https://github.com/apache/pinot/pull/9810#discussion_r1043954882
##########
pinot-core/src/test/java/org/apache/pinot/queries/ForwardIndexDisabledSingleValueQueriesTest.java:
##########
@@ -845,6 +845,69 @@ public void testSelectQueriesWithReload()
assertEquals(resultRow[0], 240528);
assertEquals(resultRow[1], "gFuH");
}
+
+ // Re-enable forward index for column9, column11, and column6
Review Comment:
done. one issue here, the MV column test is taking 7-8 minutes to run :( it
passes but maybe this is too long. Any suggestions to improve this?
Also couldn't run `SELECT *` because some other columns still have forward
index disabled in these tests. So aded a test for selecting the specific column
instead.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
