[calcite] branch master updated (a231418 -> b6d663c)

2019-03-29 Thread hongze
This is an automated email from the ASF dual-hosted git repository. hongze pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git. discard a231418 Site: update content of "Not implemented" since JSON_LENGH has already been added new b6d663c Site:

[calcite] branch master updated: Site: update content of "Not implemented" since JSON_LENGH has already been added

2019-03-29 Thread hongze
This is an automated email from the ASF dual-hosted git repository. hongze 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 a231418 Site: update content of "Not implement

[GitHub] [calcite] pengzhiwei2018 commented on a change in pull request #1138: [CALCITE-1581] UDTF like in hive

2019-03-29 Thread GitBox
pengzhiwei2018 commented on a change in pull request #1138: [CALCITE-1581] UDTF like in hive URL: https://github.com/apache/calcite/pull/1138#discussion_r270613512 ## File path: core/src/main/java/org/apache/calcite/sql/validate/SqlValidatorImpl.java ## @@ -1320,6 +1331,2

[GitHub] [calcite] asfgit closed pull request #1102: [CALCITE-1515] RelBuilder supports creating TableFunctionScan

2019-03-29 Thread GitBox
asfgit closed pull request #1102: [CALCITE-1515] RelBuilder supports creating TableFunctionScan URL: https://github.com/apache/calcite/pull/1102 This is an automated message from the Apache Git Service. To respond to the mes

[GitHub] [calcite] asfgit closed pull request #1137: [CALCITE-2928] Make UDF case-sensitiveness configurable

2019-03-29 Thread GitBox
asfgit closed pull request #1137: [CALCITE-2928] Make UDF case-sensitiveness configurable URL: https://github.com/apache/calcite/pull/1137 This is an automated message from the Apache Git Service. To respond to the message,

[calcite] 01/04: [CALCITE-1515] In RelBuilder, add functionScan method to create TableFunctionScan (Chunwei Lei)

2019-03-29 Thread jhyde
This is an automated email from the ASF dual-hosted git repository. jhyde pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git commit a3c0abd920e65afd47abc2afffd645d6f851ad15 Author: Chunwei Lei AuthorDate: Mon Mar 11 20:27:15 2019 +0800 [CALCITE-15

[calcite] 03/04: [CALCITE-2928] When resolving user-defined functions (UDFs), use the case-sensitivity of the current connection (Danny Chan)

2019-03-29 Thread jhyde
This is an automated email from the ASF dual-hosted git repository. jhyde pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git commit 136c9c5d75cf7b0809c3bde3525956d2c58470c7 Author: yuzhao.cyz AuthorDate: Thu Mar 28 14:04:03 2019 +0800 [CALCITE-292

[GitHub] [calcite] asfgit closed pull request #1110: [CALCITE-2920] RelBuilder: new method to create an antijoin (Ruben Quesada Lopez)

2019-03-29 Thread GitBox
asfgit closed pull request #1110: [CALCITE-2920] RelBuilder: new method to create an antijoin (Ruben Quesada Lopez) URL: https://github.com/apache/calcite/pull/1110 This is an automated message from the Apache Git Service. T

[calcite] 02/04: [CALCITE-2920] In RelBuilder, add antiJoin method (Ruben Quesada Lopez)

2019-03-29 Thread jhyde
This is an automated email from the ASF dual-hosted git repository. jhyde pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git commit 22fd34f892a2232d393bd5906f8bb1f21f1dff50 Author: rubenada AuthorDate: Fri Mar 15 16:56:54 2019 +0100 [CALCITE-2920]

[calcite] branch master updated (1018d25 -> 9bce62c)

2019-03-29 Thread jhyde
This is an automated email from the ASF dual-hosted git repository. jhyde pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git. from 1018d25 [CALCITE-2900] RelStructuredTypeFlattener generates wrong types on nested columns (Will Yu) new a3c0ab

[calcite] 04/04: Site: Improve documentation for MySQL-specific JSON operators

2019-03-29 Thread jhyde
This is an automated email from the ASF dual-hosted git repository. jhyde pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git commit 9bce62c6b6abd13d551d0eed1f1e14b024bbb2ae Author: Hongze Zhang AuthorDate: Mon Mar 11 15:46:08 2019 +0800 Site: Impr

[GitHub] [calcite] asfgit closed pull request #1104: [CALCITE-2900] RelStructuredTypeFlattener generates wrong types on nested columns

2019-03-29 Thread GitBox
asfgit closed pull request #1104: [CALCITE-2900] RelStructuredTypeFlattener generates wrong types on nested columns URL: https://github.com/apache/calcite/pull/1104 This is an automated message from the Apache Git Service. T

[calcite] branch master updated: [CALCITE-2900] RelStructuredTypeFlattener generates wrong types on nested columns (Will Yu)

2019-03-29 Thread zabetak
This is an automated email from the ASF dual-hosted git repository. zabetak 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 1018d25 [CALCITE-2900] RelStructuredTypeFlatt

[calcite] branch master updated: [CALCITE-2941] EnumerableLimitRule on Sort with no collation creates EnumerableLimit with wrong traitSet and cluster (Ruben Quesada Lopez)

2019-03-29 Thread sereda
This is an automated email from the ASF dual-hosted git repository. sereda 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 382f62c [CALCITE-2941] EnumerableLimitRule on

[GitHub] [calcite] asereda-gs merged pull request #1122: [CALCITE-2941] EnumerableLimitRule on Sort with no collation creates EnumerableLimit with wrong traitSet and cluster

2019-03-29 Thread GitBox
asereda-gs merged pull request #1122: [CALCITE-2941] EnumerableLimitRule on Sort with no collation creates EnumerableLimit with wrong traitSet and cluster URL: https://github.com/apache/calcite/pull/1122 This is an automated

[GitHub] [calcite] asereda-gs commented on issue #1101: [CALCITE-2909] Optimize Enumerable SemiJoin with lazy computation of innerLookup (Ruben Quesada Lopez)

2019-03-29 Thread GitBox
asereda-gs commented on issue #1101: [CALCITE-2909] Optimize Enumerable SemiJoin with lazy computation of innerLookup (Ruben Quesada Lopez) URL: https://github.com/apache/calcite/pull/1101#issuecomment-478161905 Thank you! Th

[GitHub] [calcite] asereda-gs merged pull request #1101: [CALCITE-2909] Optimize Enumerable SemiJoin with lazy computation of innerLookup (Ruben Quesada Lopez)

2019-03-29 Thread GitBox
asereda-gs merged pull request #1101: [CALCITE-2909] Optimize Enumerable SemiJoin with lazy computation of innerLookup (Ruben Quesada Lopez) URL: https://github.com/apache/calcite/pull/1101 This is an automated message from

[GitHub] [calcite] asereda-gs commented on issue #1122: [CALCITE-2941] EnumerableLimitRule on Sort with no collation creates EnumerableLimit with wrong traitSet and cluster

2019-03-29 Thread GitBox
asereda-gs commented on issue #1122: [CALCITE-2941] EnumerableLimitRule on Sort with no collation creates EnumerableLimit with wrong traitSet and cluster URL: https://github.com/apache/calcite/pull/1122#issuecomment-478162063 Thank you! -

[calcite] branch master updated: [CALCITE-2909] Optimize Enumerable SemiJoin with lazy computation of innerLookup (Ruben Quesada Lopez)

2019-03-29 Thread sereda
This is an automated email from the ASF dual-hosted git repository. sereda 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 59aa115 [CALCITE-2909] Optimize Enumerable Sem

[GitHub] [calcite] risdenk merged pull request #1139: [CALCITE-2961] Enable Travis to test against JDK 13

2019-03-29 Thread GitBox
risdenk merged pull request #1139: [CALCITE-2961] Enable Travis to test against JDK 13 URL: https://github.com/apache/calcite/pull/1139 This is an automated message from the Apache Git Service. To respond to the message, ple

[calcite] branch master updated: [CALCITE-2961] Enable Travis to test against JDK 13

2019-03-29 Thread krisden
This is an automated email from the ASF dual-hosted git repository. krisden 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 5cdf856 [CALCITE-2961] Enable Travis to test

[GitHub] [calcite] asereda-gs commented on issue #1139: [CALCITE-2961] Enable Travis to test against JDK 13

2019-03-29 Thread GitBox
asereda-gs commented on issue #1139: [CALCITE-2961] Enable Travis to test against JDK 13 URL: https://github.com/apache/calcite/pull/1139#issuecomment-47816 LGTM This is an automated message from the Apache Git Service. T

[GitHub] [calcite] risdenk commented on issue #1139: [CALCITE-2961] Enable Travis to test against JDK 13

2019-03-29 Thread GitBox
risdenk commented on issue #1139: [CALCITE-2961] Enable Travis to test against JDK 13 URL: https://github.com/apache/calcite/pull/1139#issuecomment-478151902 Pushed change to better handle checking for JDK 11+ based on review comments. --

[GitHub] [calcite] risdenk commented on a change in pull request #1139: [CALCITE-2961] Enable Travis to test against JDK 13

2019-03-29 Thread GitBox
risdenk commented on a change in pull request #1139: [CALCITE-2961] Enable Travis to test against JDK 13 URL: https://github.com/apache/calcite/pull/1139#discussion_r270575329 ## File path: cassandra/src/test/java/org/apache/calcite/test/CassandraAdapterTest.java ## @@ -8

[GitHub] [calcite] risdenk commented on issue #1139: [CALCITE-2961] Enable Travis to test against JDK 13

2019-03-29 Thread GitBox
risdenk commented on issue #1139: [CALCITE-2961] Enable Travis to test against JDK 13 URL: https://github.com/apache/calcite/pull/1139#issuecomment-478151028 Ok so build succeeds after fixing the Cassandra issue. I'll address the review comments next. -

[GitHub] [calcite] risdenk commented on a change in pull request #1139: [CALCITE-2961] Enable Travis to test against JDK 13

2019-03-29 Thread GitBox
risdenk commented on a change in pull request #1139: [CALCITE-2961] Enable Travis to test against JDK 13 URL: https://github.com/apache/calcite/pull/1139#discussion_r270574608 ## File path: cassandra/src/test/java/org/apache/calcite/test/CassandraAdapterTest.java ## @@ -8

[GitHub] [calcite] asereda-gs commented on a change in pull request #1139: [CALCITE-2961] Enable Travis to test against JDK 13

2019-03-29 Thread GitBox
asereda-gs commented on a change in pull request #1139: [CALCITE-2961] Enable Travis to test against JDK 13 URL: https://github.com/apache/calcite/pull/1139#discussion_r270574346 ## File path: cassandra/src/test/java/org/apache/calcite/test/CassandraAdapterTest.java ## @@

[calcite-avatica] branch master updated: [CALCITE-2961] Enable Travis to test against JDK 13

2019-03-29 Thread krisden
This is an automated email from the ASF dual-hosted git repository. krisden pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite-avatica.git The following commit(s) were added to refs/heads/master by this push: new 1f8a213 [CALCITE-2961] Enable Travis

[GitHub] [calcite-avatica] risdenk merged pull request #91: [CALCITE-2961] Enable Travis to test against JDK 13

2019-03-29 Thread GitBox
risdenk merged pull request #91: [CALCITE-2961] Enable Travis to test against JDK 13 URL: https://github.com/apache/calcite-avatica/pull/91 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [calcite-avatica] risdenk commented on issue #91: [CALCITE-2961] Enable Travis to test against JDK 13

2019-03-29 Thread GitBox
risdenk commented on issue #91: [CALCITE-2961] Enable Travis to test against JDK 13 URL: https://github.com/apache/calcite-avatica/pull/91#issuecomment-478146461 Woot Travis passed with upgraded asm for spotbugs. This is an a

[GitHub] [calcite] risdenk commented on a change in pull request #1139: [CALCITE-2961] Enable Travis to test against JDK 13

2019-03-29 Thread GitBox
risdenk commented on a change in pull request #1139: [CALCITE-2961] Enable Travis to test against JDK 13 URL: https://github.com/apache/calcite/pull/1139#discussion_r270569245 ## File path: cassandra/src/test/java/org/apache/calcite/test/CassandraAdapterTest.java ## @@ -8

[GitHub] [calcite] asereda-gs commented on a change in pull request #1139: [CALCITE-2961] Enable Travis to test against JDK 13

2019-03-29 Thread GitBox
asereda-gs commented on a change in pull request #1139: [CALCITE-2961] Enable Travis to test against JDK 13 URL: https://github.com/apache/calcite/pull/1139#discussion_r270568993 ## File path: cassandra/src/test/java/org/apache/calcite/test/CassandraAdapterTest.java ## @@

[GitHub] [calcite] risdenk commented on issue #1139: [CALCITE-2961] Enable Travis to test against JDK 13

2019-03-29 Thread GitBox
risdenk commented on issue #1139: [CALCITE-2961] Enable Travis to test against JDK 13 URL: https://github.com/apache/calcite/pull/1139#issuecomment-478140557 Well screwed up that logic. Fixing in a few minutes. This is an aut

[GitHub] [calcite-avatica] risdenk commented on issue #91: [CALCITE-2961] Enable Travis to test against JDK 13

2019-03-29 Thread GitBox
risdenk commented on issue #91: [CALCITE-2961] Enable Travis to test against JDK 13 URL: https://github.com/apache/calcite-avatica/pull/91#issuecomment-478135437 Pushed a change that upgrades asm for Spotbugs. This is an auto

[GitHub] [calcite-avatica] risdenk commented on issue #91: [CALCITE-2961] Enable Travis to test against JDK 13

2019-03-29 Thread GitBox
risdenk commented on issue #91: [CALCITE-2961] Enable Travis to test against JDK 13 URL: https://github.com/apache/calcite-avatica/pull/91#issuecomment-478134706 This was fixed upstream in commit - https://github.com/spotbugs/spotbugs/commit/5153f67498394fcbbbe49f331229f24b74faf58b

[GitHub] [calcite] risdenk commented on issue #1139: [CALCITE-2961] Enable Travis to test against JDK 13

2019-03-29 Thread GitBox
risdenk commented on issue #1139: [CALCITE-2961] Enable Travis to test against JDK 13 URL: https://github.com/apache/calcite/pull/1139#issuecomment-478133125 Pushed change that should skip on JDK11+ for Cassandra. We had to do the same thing on JDK 12 too.

[calcite] branch site updated: Site: Improve documentation for MySQL-specific JSON operators (does not include JSON_LENGTH)

2019-03-29 Thread jhyde
This is an automated email from the ASF dual-hosted git repository. jhyde pushed a commit to branch site in repository https://gitbox.apache.org/repos/asf/calcite.git The following commit(s) were added to refs/heads/site by this push: new aa025d2 Site: Improve documentation for MySQL-speci

[GitHub] [calcite] risdenk edited a comment on issue #1139: [CALCITE-2961] Enable Travis to test against JDK 13

2019-03-29 Thread GitBox
risdenk edited a comment on issue #1139: [CALCITE-2961] Enable Travis to test against JDK 13 URL: https://github.com/apache/calcite/pull/1139#issuecomment-478131066 Failed on Cassandra on JDK 13: ``` [INFO] Running org.apache.calcite.test.CassandraAdapterTest [[1;

[GitHub] [calcite] risdenk commented on issue #1139: [CALCITE-2961] Enable Travis to test against JDK 13

2019-03-29 Thread GitBox
risdenk commented on issue #1139: [CALCITE-2961] Enable Travis to test against JDK 13 URL: https://github.com/apache/calcite/pull/1139#issuecomment-478131066 Failed on Cassandra: ``` [INFO] Running org.apache.calcite.test.CassandraAdapterTest [ERROR] [1;

[calcite-site] branch master updated: Site: Improve documentation for MySQL-specific JSON operators (not including JSON_LENGTH)

2019-03-29 Thread jhyde
This is an automated email from the ASF dual-hosted git repository. jhyde pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite-site.git The following commit(s) were added to refs/heads/master by this push: new 4b85216 Site: Improve documentation for My

[GitHub] [calcite-avatica] risdenk edited a comment on issue #91: [CALCITE-2961] Enable Travis to test against JDK 13

2019-03-29 Thread GitBox
risdenk edited a comment on issue #91: [CALCITE-2961] Enable Travis to test against JDK 13 URL: https://github.com/apache/calcite-avatica/pull/91#issuecomment-478129247 Hmmm JDK 13 fails due to what looks like an older asm version for spotbugs. We should be able to override that. ``

[GitHub] [calcite-avatica] risdenk commented on issue #91: [CALCITE-2961] Enable Travis to test against JDK 13

2019-03-29 Thread GitBox
risdenk commented on issue #91: [CALCITE-2961] Enable Travis to test against JDK 13 URL: https://github.com/apache/calcite-avatica/pull/91#issuecomment-478129247 Hmmm JDK 13 fails due to what looks like an older ASM version. We should be able to override that.

[GitHub] [calcite] risdenk opened a new pull request #1139: [CALCITE-2961] Enable Travis to test against JDK 13

2019-03-29 Thread GitBox
risdenk opened a new pull request #1139: [CALCITE-2961] Enable Travis to test against JDK 13 URL: https://github.com/apache/calcite/pull/1139 This is an automated message from the Apache Git Service. To respond to the messag

[GitHub] [calcite-avatica] risdenk opened a new pull request #91: [CALCITE-2961] Enable Travis to test against JDK 13

2019-03-29 Thread GitBox
risdenk opened a new pull request #91: [CALCITE-2961] Enable Travis to test against JDK 13 URL: https://github.com/apache/calcite-avatica/pull/91 This is an automated message from the Apache Git Service. To respond to the me

[GitHub] [calcite] julianhyde commented on issue #1100: Site: Improve documentation for MySQL Specific JSON operators

2019-03-29 Thread GitBox
julianhyde commented on issue #1100: Site: Improve documentation for MySQL Specific JSON operators URL: https://github.com/apache/calcite/pull/1100#issuecomment-478089536 Yes, splitting into two changes would be useful. Thanks for suggesting it.

[GitHub] [calcite] XuQianJin-Stars edited a comment on issue #1086: [CALCITE-2892] Add the JSON_KEYS function

2019-03-29 Thread GitBox
XuQianJin-Stars edited a comment on issue #1086: [CALCITE-2892] Add the JSON_KEYS function URL: https://github.com/apache/calcite/pull/1086#issuecomment-477826322 hi @michaelmior @zhztheplayer I have updated this PR. Can you continue to review this PR? best qianjin -

[GitHub] [calcite] yuqi1129 commented on a change in pull request #1138: [CALCITE-1581] UDTF like in hive

2019-03-29 Thread GitBox
yuqi1129 commented on a change in pull request #1138: [CALCITE-1581] UDTF like in hive URL: https://github.com/apache/calcite/pull/1138#discussion_r270410362 ## File path: core/src/main/java/org/apache/calcite/sql/validate/SqlValidatorImpl.java ## @@ -1320,6 +1331,214 @@

[GitHub] [calcite] yuqi1129 commented on a change in pull request #1138: [CALCITE-1581] UDTF like in hive

2019-03-29 Thread GitBox
yuqi1129 commented on a change in pull request #1138: [CALCITE-1581] UDTF like in hive URL: https://github.com/apache/calcite/pull/1138#discussion_r270392435 ## File path: core/src/main/java/org/apache/calcite/runtime/CalciteResource.java ## @@ -876,6 +876,16 @@ @Base

[GitHub] [calcite] yuqi1129 commented on a change in pull request #1138: [CALCITE-1581] UDTF like in hive

2019-03-29 Thread GitBox
yuqi1129 commented on a change in pull request #1138: [CALCITE-1581] UDTF like in hive URL: https://github.com/apache/calcite/pull/1138#discussion_r270405037 ## File path: core/src/main/java/org/apache/calcite/sql/validate/SqlValidatorImpl.java ## @@ -1320,6 +1331,214 @@

[GitHub] [calcite] zabetak commented on a change in pull request #1130: [CALCITE-2865] FilterProjectTransposeRule generates wrong traitSet when copyFilter/Project is true

2019-03-29 Thread GitBox
zabetak commented on a change in pull request #1130: [CALCITE-2865] FilterProjectTransposeRule generates wrong traitSet when copyFilter/Project is true URL: https://github.com/apache/calcite/pull/1130#discussion_r270352233 ## File path: core/src/main/java/org/apache/calcite/rel/r

[GitHub] [calcite] zabetak commented on issue #1033: [CALCITE-2820] Avoid reducing certain aggregate functions when it is not necessary

2019-03-29 Thread GitBox
zabetak commented on issue #1033: [CALCITE-2820] Avoid reducing certain aggregate functions when it is not necessary URL: https://github.com/apache/calcite/pull/1033#issuecomment-477944403 Conflicts have to be resolved first!

[GitHub] [calcite] XuQianJin-Stars edited a comment on issue #1086: [CALCITE-2892] Add the JSON_KEYS function

2019-03-29 Thread GitBox
XuQianJin-Stars edited a comment on issue #1086: [CALCITE-2892] Add the JSON_KEYS function URL: https://github.com/apache/calcite/pull/1086#issuecomment-477826322 hi @michaelmior @zhztheplayer I have updated this PR. Can you continue to review this PR? best qianjin

[GitHub] [calcite] rubenada commented on a change in pull request #1130: [CALCITE-2865] FilterProjectTransposeRule generates wrong traitSet when copyFilter/Project is true

2019-03-29 Thread GitBox
rubenada commented on a change in pull request #1130: [CALCITE-2865] FilterProjectTransposeRule generates wrong traitSet when copyFilter/Project is true URL: https://github.com/apache/calcite/pull/1130#discussion_r270310144 ## File path: core/src/main/java/org/apache/calci

[GitHub] [calcite-avatica] zabetak commented on a change in pull request #85: [CALCITE-2704] Avoid use of ISO-8859-1 to parse request in JsonHandler

2019-03-29 Thread GitBox
zabetak commented on a change in pull request #85: [CALCITE-2704] Avoid use of ISO-8859-1 to parse request in JsonHandler URL: https://github.com/apache/calcite-avatica/pull/85#discussion_r270308361 ## File path: server/src/main/java/org/apache/calcite/avatica/server/AvaticaJsonHan

[GitHub] [calcite-avatica] zabetak commented on a change in pull request #85: [CALCITE-2704] Avoid use of ISO-8859-1 to parse request in JsonHandler

2019-03-29 Thread GitBox
zabetak commented on a change in pull request #85: [CALCITE-2704] Avoid use of ISO-8859-1 to parse request in JsonHandler URL: https://github.com/apache/calcite-avatica/pull/85#discussion_r270307873 ## File path: server/src/main/java/org/apache/calcite/avatica/server/AvaticaJsonHan