albertobastos commented on code in PR #14830:
URL: https://github.com/apache/pinot/pull/14830#discussion_r1925347842
##########
pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/MultiStageEngineIntegrationTest.java:
##########
@@ -1347,6 +1347,15 @@ public void testConcurrentQueries() {
executorService.shutdownNow();
}
+ @Test
+ public void testCaseInsensitiveNames() throws Exception {
+ String query = "select ACTualELAPsedTIMe from mYtABLE where
actUALelAPSedTIMe > 0 limit 1";
+ JsonNode jsonNode = postQuery(query);
Review Comment:
Easy-peasy.
https://github.com/apache/pinot/pull/14830/commits/f512e2b830785132ec9e178235930e108ca24cbf
I didn't do that because each other test within this file is only querying
against the Broker API, should we also de-duplicate them?
--
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
