[GitHub] [druid] paul-rogers commented on a diff in pull request #13844: Change test name so it triggers in CI

["paul-rogers (via GitHub)" <gi...@apache.org>]

paul-rogers commented on code in PR #13844:
URL: https://github.com/apache/druid/pull/13844#discussion_r1117805526


##########
extensions-core/multi-stage-query/src/test/java/org/apache/druid/msq/test/CalciteSelectQueryMSQTest.java:
##########
@@ -161,4 +165,19 @@ public void testQueryWithMoreThanMaxNumericInFilter()
 
   }
 
+  @Test
+  @Override
+  public void testArrayAggQueryOnComplexDatatypes()

Review Comment:
   Would be worth including an explanation that this feature does not work on MSQ, but does work with "classic" queries. Might be wroth also adding links between the tests to cross-reference the MSQ vs. "classic" case.
   
   That way, if this ever changes in the future, folks can sort out where the two copies of the test live.



-- 
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-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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