[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] xy2953396112 commented on a change in pull request #2021: [CALCITE-3929] When deserialize UDAF aggregate call from json string,…

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

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

2020-06-22 Thread GitBox
xy2953396112 commented on a change in pull request #2021: URL: https://github.com/apache/calcite/pull/2021#discussion_r443685867 ## File path: core/src/test/java/org/apache/calcite/plan/RelWriterTest.java ## @@ -146,8 +147,9 @@ + "{\n" + "

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

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

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

2020-06-22 Thread GitBox
xy2953396112 commented on a change in pull request #2021: URL: https://github.com/apache/calcite/pull/2021#discussion_r443681619 ## File path: core/src/main/java/org/apache/calcite/rel/externalize/RelJson.java ## @@ -271,7 +271,9 @@ public RelDataType toType(RelDataTypeFactory

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

2020-06-19 Thread GitBox
xy2953396112 commented on a change in pull request #2021: URL: https://github.com/apache/calcite/pull/2021#discussion_r443085303 ## File path: core/src/main/java/org/apache/calcite/rel/externalize/RelJson.java ## @@ -271,7 +271,9 @@ public RelDataType toType(RelDataTypeFactory

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

2020-06-19 Thread GitBox
xy2953396112 commented on a change in pull request #2021: URL: https://github.com/apache/calcite/pull/2021#discussion_r443085303 ## File path: core/src/main/java/org/apache/calcite/rel/externalize/RelJson.java ## @@ -271,7 +271,9 @@ public RelDataType toType(RelDataTypeFactory