mihaibudiu commented on code in PR #4949:
URL: https://github.com/apache/calcite/pull/4949#discussion_r3269473079


##########
core/src/test/java/org/apache/calcite/test/MaterializedViewRelOptRulesTest.java:
##########
@@ -251,6 +251,44 @@ protected final MaterializedViewFixture sql(String 
materialize,
         .ok();
   }
 
+  /** Test case for <a 
href="https://issues.apache.org/jira/browse/CALCITE-7534";>[CALCITE-7534]
+   * Support GROUPING SETS, CUBE and ROLLUP in materialized view aggregate 
rule rewriting</a>. */
+  @Test void testAggregateGroupSets1() {

Review Comment:
   By reading this test I cannot tell why it is related to materialized view 
aggregate rewriting.
   Could the test contain two versions of the expected program: with and 
without the rewriting?



-- 
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]

Reply via email to