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

2019-06-05 Thread GitBox
siddharthteotia 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_r290894078 ## File path: core/src/main/java/org/apache/calcite/rel/core/RelFactories.java ##

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

2019-06-05 Thread GitBox
siddharthteotia 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_r290894027 ## File path: core/src/main/java/org/apache/calcite/tools/RelBuilder.java ## @@

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

2019-06-05 Thread GitBox
siddharthteotia 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_r290892445 ## File path: core/src/main/java/org/apache/calcite/adapter/enumerable/EnumerableRel.java

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

2019-06-05 Thread GitBox
siddharthteotia 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_r290891898 ## File path: core/src/test/java/org/apache/calcite/test/RelOptRulesTest.java ##

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

2019-04-02 Thread GitBox
siddharthteotia 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_r271498915 ## File path: core/src/test/java/org/apache/calcite/test/RelOptRulesTest.java ##

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

2019-04-02 Thread GitBox
siddharthteotia 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_r271498915 ## File path: core/src/test/java/org/apache/calcite/test/RelOptRulesTest.java ##

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

2019-04-02 Thread GitBox
siddharthteotia 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_r271498570 ## File path: core/src/main/java/org/apache/calcite/rel/rules/FilterMultiJoinMergeRule.ja

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

2019-04-02 Thread GitBox
siddharthteotia 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_r271485479 ## File path: core/src/main/java/org/apache/calcite/rel/rules/FilterMultiJoinMergeRule.ja

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

2019-04-02 Thread GitBox
siddharthteotia 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_r271485206 ## File path: core/src/main/java/org/apache/calcite/plan/RelOptUtil.java ## @@ -

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

2019-03-12 Thread GitBox
siddharthteotia 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_r264793810 ## File path: core/src/test/java/org/apache/calcite/test/RelOptRulesTest.java ##