[GitHub] kgyrtkirk commented on a change in pull request #1056: CALCITE-2858

2019-02-22 Thread GitBox
kgyrtkirk commented on a change in pull request #1056: CALCITE-2858 URL: https://github.com/apache/calcite/pull/1056#discussion_r259429721 ## File path: core/src/main/java/org/apache/calcite/rel/externalize/RelJson.java ## @@ -391,13 +385,17 @@ RexNode toRex(RelInput

[GitHub] kgyrtkirk commented on a change in pull request #1056: CALCITE-2858

2019-02-21 Thread GitBox
kgyrtkirk commented on a change in pull request #1056: CALCITE-2858 URL: https://github.com/apache/calcite/pull/1056#discussion_r258865526 ## File path: core/src/main/java/org/apache/calcite/rel/externalize/RelJson.java ## @@ -391,13 +385,17 @@ RexNode toRex(RelInput