rubenada commented on a change in pull request #1307: [CALCITE-3182] Trim
unused fields for plan of materialized-view before matching.
URL: https://github.com/apache/calcite/pull/1307#discussion_r301117965
##########
File path: core/src/test/java/org/apache/calcite/test/MaterializationTest.java
##########
@@ -2389,6 +2389,19 @@ private void checkSatisfiable(RexNode e, String s) {
}
}
+ @Test public void testMaterializationAfterTrimingOfUnusedFields() {
+ String sql =
+ "select \"y\".\"deptno\", \"y\".\"name\", \"x\".\"sum_salary\"\n" +
Review comment:
It seems that there are checkstyle violations in this method.
Please, before submitting a change, verify that the build passes locally
using `mvn clean install`.
----------------------------------------------------------------
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
