[calcite] branch master updated: [CALCITE-4884] Provide a new constructor for RelJsonWriter to allow customized JsonBuilder

2021-11-16 Thread chunwei
This is an automated email from the ASF dual-hosted git repository. chunwei 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 faee09f [CALCITE-4884] Provide a new construc

[GitHub] [calcite] chunweilei merged pull request #2610: [CALCITE-4884] Provide a new constructor for RelJsonWriter to allow customized JsonBuilder

2021-11-16 Thread GitBox
chunweilei merged pull request #2610: URL: https://github.com/apache/calcite/pull/2610 -- 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 to the specific comment. To unsubscribe, e-mail: commits-uns

[GitHub] [calcite] chunweilei commented on pull request #2610: [CALCITE-4884] Provide a new constructor for RelJsonWriter to allow customized JsonBuilder

2021-11-16 Thread GitBox
chunweilei commented on pull request #2610: URL: https://github.com/apache/calcite/pull/2610#issuecomment-971260207 > ustomized --> customized? Oops. The title is wrong while the commit message is right. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [calcite] xy2953396112 commented on pull request #2610: [CALCITE-4884] Provide a new constructor for RelJsonWriter to allow ustomized JsonBuilder

2021-11-16 Thread GitBox
xy2953396112 commented on pull request #2610: URL: https://github.com/apache/calcite/pull/2610#issuecomment-970371582 LGTM~ -- 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 to the specific comment.

[GitHub] [calcite] xy2953396112 commented on pull request #2610: [CALCITE-4884] Provide a new constructor for RelJsonWriter to allow ustomized JsonBuilder

2021-11-16 Thread GitBox
xy2953396112 commented on pull request #2610: URL: https://github.com/apache/calcite/pull/2610#issuecomment-970333860 ustomized --> customized? -- 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 to th

[calcite] branch master updated (07e420b -> 0899e6c)

2021-11-16 Thread xiong
This is an automated email from the ASF dual-hosted git repository. xiong pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git. from 07e420b [CALCITE-4877] Enable schema.iq test, with variants depending on Avatica version add 0899e6c [CALCITE

[GitHub] [calcite] NobiGo merged pull request #2607: [CALCITE-4846] IN-list that includes NULL converted to Values throws exception

2021-11-16 Thread GitBox
NobiGo merged pull request #2607: URL: https://github.com/apache/calcite/pull/2607 -- 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 to the specific comment. To unsubscribe, e-mail: commits-unsubsc

[GitHub] [calcite] ILuffZhe commented on a change in pull request #2609: [CALCITE-4876] Converting RelNode to SQL with CalciteSqlDialect gets wrong result while EnumerableIntersect is followed by Enum

2021-11-16 Thread GitBox
ILuffZhe commented on a change in pull request #2609: URL: https://github.com/apache/calcite/pull/2609#discussion_r750309422 ## File path: core/src/test/java/org/apache/calcite/rel/rel2sql/RelToSqlConverterTest.java ## @@ -1881,6 +1881,31 @@ private void checkHavingAliasSameAs

[GitHub] [calcite] NobiGo commented on a change in pull request #2609: [CALCITE-4876] Converting RelNode to SQL with CalciteSqlDialect gets wrong result while EnumerableIntersect is followed by Enumer

2021-11-16 Thread GitBox
NobiGo commented on a change in pull request #2609: URL: https://github.com/apache/calcite/pull/2609#discussion_r750296089 ## File path: core/src/test/java/org/apache/calcite/rel/rel2sql/RelToSqlConverterTest.java ## @@ -1881,6 +1881,31 @@ private void checkHavingAliasSameAsCo

[GitHub] [calcite] vlsi commented on a change in pull request #2607: [CALCITE-4846] IN-list that includes NULL converted to Values throws exception

2021-11-16 Thread GitBox
vlsi commented on a change in pull request #2607: URL: https://github.com/apache/calcite/pull/2607#discussion_r750219731 ## File path: core/src/test/resources/sql/sub-query.iq ## @@ -3306,4 +3306,28 @@ EnumerableCalc(expr#0..7=[{inputs}], expr#8=[7782], expr#9=[CAST($t0):INTEG