[GitHub] [calcite] jcamachor commented on a change in pull request #2094: [CALCITE-3409] Add an interface in RelOptMaterializations…

jcamachor commented on a change in pull request #2094: URL: https://github.com/apache/calcite/pull/2094#discussion_r714094628 ## File path: core/src/main/java/org/apache/calcite/plan/RelOptMaterializations.java ## @@ -214,7 +232,11 @@ hepPlanner.setRoot(root); root

[GitHub] [calcite] jcamachor commented on a change in pull request #2094: [CALCITE-3409] Add an interface in RelOptMaterializations…

jcamachor commented on a change in pull request #2094: URL: https://github.com/apache/calcite/pull/2094#discussion_r516873161 ## File path: core/src/main/java/org/apache/calcite/plan/AbstractRelOptPlanner.java ## @@ -197,6 +198,10 @@ public boolean isRuleExcluded(RelOptRule

[GitHub] [calcite] jcamachor commented on a change in pull request #2094: [CALCITE-3409] Add an interface in RelOptMaterializations…

jcamachor commented on a change in pull request #2094: URL: https://github.com/apache/calcite/pull/2094#discussion_r516184828 ## File path: core/src/main/java/org/apache/calcite/plan/RelOptMaterializations.java ## @@ -50,6 +50,12 @@ */ public abstract class