suneet-amp commented on a change in pull request #8815: SQL: EARLIEST, LATEST
aggregators.
URL: https://github.com/apache/incubator-druid/pull/8815#discussion_r342235271
##########
File path: sql/src/test/java/org/apache/druid/sql/calcite/CalciteQueryTest.java
##########
@@ -889,6 +896,123 @@ public void testGroupBySingleColumnDescendingNoTopN()
throws Exception
);
}
+ @Test
+ public void testEarliestAggregators() throws Exception
+ {
+ // Cannot vectorize EARLIEST aggregator.
+ skipVectorize();
Review comment:
Does this need to be reset at the end of the test? Otherwise all the other
tests that run after this will run with vectorize disabled? Or is that being
handled by some rule that I'm not seeing in this delta
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
