[GitHub] [calcite] cuglifangzheng commented on a change in pull request #2121: [CALCITE-4192] RelMdColumnOrigins should obtain the true column index of the group by column of the aggregation operator

2020-09-12 Thread GitBox
cuglifangzheng commented on a change in pull request #2121: URL: https://github.com/apache/calcite/pull/2121#discussion_r487359768 ## File path: core/src/test/java/org/apache/calcite/test/RelOptRulesTest.java ## @@ -6764,4 +6769,25 @@ protected MyProjectRule(Config config) {

[GitHub] [calcite] cuglifangzheng commented on a change in pull request #2121: [CALCITE-4192] RelMdColumnOrigins should obtain the true column index of the group by column of the aggregation operator

2020-09-12 Thread GitBox
cuglifangzheng commented on a change in pull request #2121: URL: https://github.com/apache/calcite/pull/2121#discussion_r487359768 ## File path: core/src/test/java/org/apache/calcite/test/RelOptRulesTest.java ## @@ -6764,4 +6769,25 @@ protected MyProjectRule(Config config) {

[GitHub] [calcite] cuglifangzheng commented on a change in pull request #2121: [CALCITE-4192] RelMdColumnOrigins should obtain the true column index of the group by column of the aggregation operator

2020-09-12 Thread GitBox
cuglifangzheng commented on a change in pull request #2121: URL: https://github.com/apache/calcite/pull/2121#discussion_r487359768 ## File path: core/src/test/java/org/apache/calcite/test/RelOptRulesTest.java ## @@ -6764,4 +6769,25 @@ protected MyProjectRule(Config config) {

[GitHub] [calcite] cuglifangzheng commented on a change in pull request #2121: [CALCITE-4192] RelMdColumnOrigins should obtain the true column index of the group by column of the aggregation operator

2020-09-11 Thread GitBox
cuglifangzheng commented on a change in pull request #2121: URL: https://github.com/apache/calcite/pull/2121#discussion_r487359768 ## File path: core/src/test/java/org/apache/calcite/test/RelOptRulesTest.java ## @@ -6764,4 +6769,25 @@ protected MyProjectRule(Config config) {