[GitHub] [calcite] zabetak commented on a change in pull request #1030: [CALCITE-2822] Allow MultiJoin rules with any project/filter

2019-04-02 Thread GitBox
zabetak commented on a change in pull request #1030: [CALCITE-2822] Allow MultiJoin rules with any project/filter URL: https://github.com/apache/calcite/pull/1030#discussion_r271510539 ## File path: core/src/main/java/org/apache/calcite/adapter/enumerable/EnumerableRel.java ##

[GitHub] [calcite] zabetak commented on a change in pull request #1030: [CALCITE-2822] Allow MultiJoin rules with any project/filter

2019-04-02 Thread GitBox
zabetak commented on a change in pull request #1030: [CALCITE-2822] Allow MultiJoin rules with any project/filter URL: https://github.com/apache/calcite/pull/1030#discussion_r271511373 ## File path: core/src/main/java/org/apache/calcite/tools/RelBuilder.java ## @@ -2623,6

[GitHub] [calcite] zabetak commented on a change in pull request #1030: [CALCITE-2822] Allow MultiJoin rules with any project/filter

2019-04-02 Thread GitBox
zabetak commented on a change in pull request #1030: [CALCITE-2822] Allow MultiJoin rules with any project/filter URL: https://github.com/apache/calcite/pull/1030#discussion_r271510854 ## File path: core/src/main/java/org/apache/calcite/rel/core/RelFactories.java ## @@ -29

[GitHub] [calcite] zabetak commented on a change in pull request #1030: [CALCITE-2822] Allow MultiJoin rules with any project/filter

2019-04-02 Thread GitBox
zabetak commented on a change in pull request #1030: [CALCITE-2822] Allow MultiJoin rules with any project/filter URL: https://github.com/apache/calcite/pull/1030#discussion_r271511665 ## File path: core/src/test/java/org/apache/calcite/test/RelOptRulesTest.java ## @@ -432

[GitHub] [calcite] zabetak commented on a change in pull request #1030: [CALCITE-2822] Allow MultiJoin rules with any project/filter

2019-04-01 Thread GitBox
zabetak commented on a change in pull request #1030: [CALCITE-2822] Allow MultiJoin rules with any project/filter URL: https://github.com/apache/calcite/pull/1030#discussion_r271079033 ## File path: core/src/main/java/org/apache/calcite/rel/rules/FilterMultiJoinMergeRule.java

[GitHub] [calcite] zabetak commented on a change in pull request #1030: [CALCITE-2822] Allow MultiJoin rules with any project/filter

2019-04-01 Thread GitBox
zabetak commented on a change in pull request #1030: [CALCITE-2822] Allow MultiJoin rules with any project/filter URL: https://github.com/apache/calcite/pull/1030#discussion_r271080751 ## File path: core/src/test/java/org/apache/calcite/test/RelOptRulesTest.java ## @@ -432

[GitHub] [calcite] zabetak commented on a change in pull request #1030: [CALCITE-2822] Allow MultiJoin rules with any project/filter

2019-04-01 Thread GitBox
zabetak commented on a change in pull request #1030: [CALCITE-2822] Allow MultiJoin rules with any project/filter URL: https://github.com/apache/calcite/pull/1030#discussion_r271078238 ## File path: core/src/main/java/org/apache/calcite/plan/RelOptUtil.java ## @@ -2611,12

[GitHub] [calcite] zabetak commented on a change in pull request #1030: [CALCITE-2822] Allow MultiJoin rules with any project/filter

2019-04-01 Thread GitBox
zabetak commented on a change in pull request #1030: [CALCITE-2822] Allow MultiJoin rules with any project/filter URL: https://github.com/apache/calcite/pull/1030#discussion_r271078470 ## File path: core/src/main/java/org/apache/calcite/rel/rules/FilterMultiJoinMergeRule.java