[GitHub] [calcite] yanlin-Lynn commented on pull request #2030: minor, remove redundant code in "RelJson.toJson(RelDistribution)".

2020-06-27 Thread GitBox
yanlin-Lynn commented on pull request #2030: URL: https://github.com/apache/calcite/pull/2030#issuecomment-650521380 you are changing the code, worth a jira for this PR. This is an automated message from the Apache Git

[calcite] branch master updated: [CALCITE-4062] Support deserialize UDF array type from json string (Xu Zhaohui)

2020-06-27 Thread yanlin
This is an automated email from the ASF dual-hosted git repository. yanlin pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git The following commit(s) were added to refs/heads/master by this push: new d8e5590 [CALCITE-4062] Support deserialize

[GitHub] [calcite] yanlin-Lynn merged pull request #2022: [CALCITE-4062] Support deserialize UDF array type from json string

2020-06-27 Thread GitBox
yanlin-Lynn merged pull request #2022: URL: https://github.com/apache/calcite/pull/2022 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

[GitHub] [calcite] yanlin-Lynn commented on a change in pull request #2013: [CALCITE-3935] Materialization-Failed, when querying with LeftJoinWithFilter

2020-06-27 Thread GitBox
yanlin-Lynn commented on a change in pull request #2013: URL: https://github.com/apache/calcite/pull/2013#discussion_r446513799 ## File path: core/src/main/java/org/apache/calcite/plan/SubstitutionVisitor.java ## @@ -2011,6 +2006,38 @@ public static boolean equalType(String

[GitHub] [calcite] hsyuan commented on pull request #2035: [CALCITE-4008] Implement Code generation for EnumerableSortedAggregat…

2020-06-27 Thread GitBox
hsyuan commented on pull request #2035: URL: https://github.com/apache/calcite/pull/2035#issuecomment-650560583 @rubenada Thanks for your help. This is an automated message from the Apache Git Service. To respond to the

[GitHub] [calcite] xy2953396112 commented on a change in pull request #2021: [CALCITE-3929] When deserialize UDAF aggregate call from json string,…

2020-06-27 Thread GitBox
xy2953396112 commented on a change in pull request #2021: URL: https://github.com/apache/calcite/pull/2021#discussion_r446503752 ## File path: core/src/test/java/org/apache/calcite/test/MockSqlOperatorTable.java ## @@ -125,6 +125,18 @@ public RelDataType

[GitHub] [calcite] hsyuan edited a comment on pull request #2035: [CALCITE-4008] Implement Code generation for EnumerableSortedAggregat…

2020-06-27 Thread GitBox
hsyuan edited a comment on pull request #2035: URL: https://github.com/apache/calcite/pull/2035#issuecomment-650560583 @rubenada Thanks for your help. As long as you approved on it, we are good to go. This is an automated

[GitHub] [calcite] yanlin-Lynn commented on a change in pull request #2013: [CALCITE-3935] Materialization-Failed, when querying with LeftJoinWithFilter

2020-06-27 Thread GitBox
yanlin-Lynn commented on a change in pull request #2013: URL: https://github.com/apache/calcite/pull/2013#discussion_r446513643 ## File path: core/src/main/java/org/apache/calcite/plan/SubstitutionVisitor.java ## @@ -2011,6 +2006,38 @@ public static boolean equalType(String

[calcite] branch master updated (d8e5590 -> da89b41)

2020-06-27 Thread chunwei
This is an automated email from the ASF dual-hosted git repository. chunwei pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git. from d8e5590 [CALCITE-4062] Support deserialize UDF array type from json string (Xu Zhaohui) add da89b41 Site:

[GitHub] [calcite] chunweilei merged pull request #1964: Site: Add Tencent and TBDS logo in powered-by image

2020-06-27 Thread GitBox
chunweilei merged pull request #1964: URL: https://github.com/apache/calcite/pull/1964 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

[GitHub] [calcite] danny0405 closed pull request #1169: [CALCITE-2948] SqlToRelConverter generates complicated logical plan f…

2020-06-27 Thread GitBox
danny0405 closed pull request #1169: URL: https://github.com/apache/calcite/pull/1169 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

[GitHub] [calcite] liyafan82 commented on a change in pull request #2027: [CALCITE-4049] Improve the implementation of the shortest-path algorithm

2020-06-27 Thread GitBox
liyafan82 commented on a change in pull request #2027: URL: https://github.com/apache/calcite/pull/2027#discussion_r446590442 ## File path: core/src/test/java/org/apache/calcite/util/graph/DirectedGraphTest.java ## @@ -64,14 +64,21 @@ g.addEdge("B", "D");

[GitHub] [calcite] danny0405 closed pull request #1279: [CALCITE-3128] Joining two tables producing only NULLs will return 0 …

2020-06-27 Thread GitBox
danny0405 closed pull request #1279: URL: https://github.com/apache/calcite/pull/1279 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

[GitHub] [calcite] liyafan82 commented on a change in pull request #2027: [CALCITE-4049] Improve the implementation of the shortest-path algorithm

2020-06-27 Thread GitBox
liyafan82 commented on a change in pull request #2027: URL: https://github.com/apache/calcite/pull/2027#discussion_r446590701 ## File path: core/src/main/java/org/apache/calcite/util/graph/Graphs.java ## @@ -102,41 +99,28 @@ public int size() { */ public static class