rohangarg commented on code in PR #15057:
URL: https://github.com/apache/druid/pull/15057#discussion_r1345692499
##########
sql/src/test/java/org/apache/druid/sql/calcite/CalciteWindowQueryTest.java:
##########
@@ -101,6 +102,7 @@ public CalciteWindowQueryTest(
@SuppressWarnings("unchecked")
public void windowQueryTest() throws IOException
{
+ assumeTrue("These tests are only run in sqlCompatible mode!",
NullHandling.sqlCompatible());
Review Comment:
Discussed with Zoltan and Eric - we're planning to support testing window
function with default value mode as well since a lot of running deployments do
use that feature. Zoltan will put up the change for testing that in a follow
up, since that requires fixing another bug as well.
--
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]
