[GitHub] [calcite] asfgit closed pull request #1177: [CALCITE-2453] Parsing list of SQL statements separated with a semicolon

2019-04-23 Thread GitBox
asfgit closed pull request #1177: [CALCITE-2453] Parsing list of SQL statements separated with a semicolon URL: https://github.com/apache/calcite/pull/1177 This is an automated message from the Apache Git Service. To respond

[calcite] branch master updated: [CALCITE-2453] Parse list of SQL statements separated with a semicolon (Chunwei Lei, charbel yazbeck)

2019-04-23 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 The following commit(s) were added to refs/heads/master by this push: new 986a2d5 [CALCITE-2453] Parse list of SQL statem

[GitHub] [calcite] amaliujia commented on a change in pull request #1083: [CALCITE-2889] IndexOutOfBoundsException thrown if targetTypes[] contains varargs

2019-04-23 Thread GitBox
amaliujia commented on a change in pull request #1083: [CALCITE-2889] IndexOutOfBoundsException thrown if targetTypes[] contains varargs URL: https://github.com/apache/calcite/pull/1083#discussion_r277910171 ## File path: core/src/main/java/org/apache/calcite/adapter/enumerable/Enu

[GitHub] [calcite] amaliujia commented on a change in pull request #1083: [CALCITE-2889] IndexOutOfBoundsException thrown if targetTypes[] contains varargs

2019-04-23 Thread GitBox
amaliujia commented on a change in pull request #1083: [CALCITE-2889] IndexOutOfBoundsException thrown if targetTypes[] contains varargs URL: https://github.com/apache/calcite/pull/1083#discussion_r277910206 ## File path: core/src/main/java/org/apache/calcite/adapter/enumerable/Enu

[GitHub] [calcite] amaliujia commented on a change in pull request #1083: [CALCITE-2889] IndexOutOfBoundsException thrown if targetTypes[] contains varargs

2019-04-23 Thread GitBox
amaliujia commented on a change in pull request #1083: [CALCITE-2889] IndexOutOfBoundsException thrown if targetTypes[] contains varargs URL: https://github.com/apache/calcite/pull/1083#discussion_r277910171 ## File path: core/src/main/java/org/apache/calcite/adapter/enumerable/Enu

[GitHub] [calcite] amaliujia commented on a change in pull request #1083: [CALCITE-2889] IndexOutOfBoundsException thrown if targetTypes[] contains varargs

2019-04-23 Thread GitBox
amaliujia commented on a change in pull request #1083: [CALCITE-2889] IndexOutOfBoundsException thrown if targetTypes[] contains varargs URL: https://github.com/apache/calcite/pull/1083#discussion_r277909684 ## File path: core/src/main/java/org/apache/calcite/adapter/enumerable/Enu

[GitHub] [calcite] hsyuan commented on issue #691: [CALCITE-500] Fix EnumerableJoin to build the left side and probe the right

2019-04-23 Thread GitBox
hsyuan commented on issue #691: [CALCITE-500] Fix EnumerableJoin to build the left side and probe the right URL: https://github.com/apache/calcite/pull/691#issuecomment-486005157 @atris Are you going to continue working on this issue? ---

[GitHub] [calcite] hsyuan closed pull request #713: [CALCITE-2343] Fix infinite loop caused by PushProjector

2019-04-23 Thread GitBox
hsyuan closed pull request #713: [CALCITE-2343] Fix infinite loop caused by PushProjector URL: https://github.com/apache/calcite/pull/713 This is an automated message from the Apache Git Service. To respond to the message, p

[GitHub] [calcite] hsyuan commented on issue #713: [CALCITE-2343] Fix infinite loop caused by PushProjector

2019-04-23 Thread GitBox
hsyuan commented on issue #713: [CALCITE-2343] Fix infinite loop caused by PushProjector URL: https://github.com/apache/calcite/pull/713#issuecomment-486004492 This has been fixed in PR https://github.com/apache/calcite/pull/1162, so I am closing this one. Thanks!

[GitHub] [calcite] hsyuan merged pull request #1166: [CALCITE-3004] RexOver is incorrectly pushed down in ProjectSetOpTran…

2019-04-23 Thread GitBox
hsyuan merged pull request #1166: [CALCITE-3004] RexOver is incorrectly pushed down in ProjectSetOpTran… URL: https://github.com/apache/calcite/pull/1166 This is an automated message from the Apache Git Service. To respond t

[calcite] branch master updated: [CALCITE-3004] RexOver is incorrectly pushed down in ProjectSetOpTransposeRule and ProjectCorrelateTransposeRule (Chunwei Lei)

2019-04-23 Thread hyuan
This is an automated email from the ASF dual-hosted git repository. hyuan 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 f26d92f [CALCITE-3004] RexOver is incorrectly p

[GitHub] [calcite] zabetak commented on a change in pull request #1083: [CALCITE-2889] IndexOutOfBoundsException thrown if targetTypes[] contains varargs

2019-04-23 Thread GitBox
zabetak commented on a change in pull request #1083: [CALCITE-2889] IndexOutOfBoundsException thrown if targetTypes[] contains varargs URL: https://github.com/apache/calcite/pull/1083#discussion_r277806753 ## File path: core/src/main/java/org/apache/calcite/adapter/enumerable/EnumU

[GitHub] [calcite] asfgit closed pull request #1171: [CALCITE-3011] Support for joins with AggregateJoinTransposeRule

2019-04-23 Thread GitBox
asfgit closed pull request #1171: [CALCITE-3011] Support for joins with AggregateJoinTransposeRule URL: https://github.com/apache/calcite/pull/1171 This is an automated message from the Apache Git Service. To respond to the

[calcite] branch master updated: [CALCITE-3011] Support left and right outer joins with AggregateJoinTransposeRule (Vineet Garg)

2019-04-23 Thread jcamacho
This is an automated email from the ASF dual-hosted git repository. jcamacho 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 ecb469b [CALCITE-3011] Support left and righ

[GitHub] [calcite] vineetgarg02 commented on a change in pull request #1171: [CALCITE-3011] Support for joins with AggregateJoinTransposeRule

2019-04-23 Thread GitBox
vineetgarg02 commented on a change in pull request #1171: [CALCITE-3011] Support for joins with AggregateJoinTransposeRule URL: https://github.com/apache/calcite/pull/1171#discussion_r29565 ## File path: core/src/main/java/org/apache/calcite/rel/rules/AggregateJoinTransposeRule

[GitHub] [calcite] vineetgarg02 commented on a change in pull request #1171: [CALCITE-3011] Support for joins with AggregateJoinTransposeRule

2019-04-23 Thread GitBox
vineetgarg02 commented on a change in pull request #1171: [CALCITE-3011] Support for joins with AggregateJoinTransposeRule URL: https://github.com/apache/calcite/pull/1171#discussion_r29420 ## File path: core/src/main/java/org/apache/calcite/rel/rules/AggregateJoinTransposeRule

[GitHub] [calcite] jcamachor commented on a change in pull request #1171: [CALCITE-3011] Support for joins with AggregateJoinTransposeRule

2019-04-23 Thread GitBox
jcamachor commented on a change in pull request #1171: [CALCITE-3011] Support for joins with AggregateJoinTransposeRule URL: https://github.com/apache/calcite/pull/1171#discussion_r277763481 ## File path: core/src/main/java/org/apache/calcite/rel/rules/AggregateJoinTransposeRule.ja

[GitHub] [calcite] jcamachor commented on a change in pull request #1171: [CALCITE-3011] Support for joins with AggregateJoinTransposeRule

2019-04-23 Thread GitBox
jcamachor commented on a change in pull request #1171: [CALCITE-3011] Support for joins with AggregateJoinTransposeRule URL: https://github.com/apache/calcite/pull/1171#discussion_r277764298 ## File path: core/src/main/java/org/apache/calcite/rel/rules/AggregateJoinTransposeRule.ja

[calcite] branch master updated: [CALCITE-3001] Upgrade to Apache Druid 0.14.0-incubating

2019-04-23 Thread jcamacho
This is an automated email from the ASF dual-hosted git repository. jcamacho 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 898ddbd [CALCITE-3001] Upgrade to Apache Dru

[GitHub] [calcite] asfgit closed pull request #1174: [CALCITE-3001] Upgrade to Apache Druid 0.14.0-incubating

2019-04-23 Thread GitBox
asfgit closed pull request #1174: [CALCITE-3001] Upgrade to Apache Druid 0.14.0-incubating URL: https://github.com/apache/calcite/pull/1174 This is an automated message from the Apache Git Service. To respond to the message,

[calcite] branch master updated: Following [CALCITE-3010], remove redundant non-reserved keyword definitions

2019-04-23 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 79df7f9 Following [CALCITE-3010], remove redun

[GitHub] [calcite] zhztheplayer merged pull request #1159: [CALCITE-2993] ParseException may be thrown for legal SQL queries due…

2019-04-23 Thread GitBox
zhztheplayer merged pull request #1159: [CALCITE-2993] ParseException may be thrown for legal SQL queries due… URL: https://github.com/apache/calcite/pull/1159 This is an automated message from the Apache Git Service. To res

[calcite] branch master updated: [CALCITE-2993] ParseException may be thrown for legal SQL queries due to incorrect "LOOKAHEAD(1)" hints

2019-04-23 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 577e72c [CALCITE-2993] ParseException may be t

[GitHub] [calcite] chunweilei commented on issue #1164: [CALCITE-2998] RexCopier should support all rex types (Chunwei Lei)

2019-04-23 Thread GitBox
chunweilei commented on issue #1164: [CALCITE-2998] RexCopier should support all rex types (Chunwei Lei) URL: https://github.com/apache/calcite/pull/1164#issuecomment-485817334 @zabetak, I updated the PR using `super.visitWindow(window)`. Could you please review again? Thanks a lot. -

[GitHub] [calcite] nishantmonu51 commented on a change in pull request #1174: [CALCITE-3001] Upgrade to Apache Druid 0.14.0-incubating

2019-04-23 Thread GitBox
nishantmonu51 commented on a change in pull request #1174: [CALCITE-3001] Upgrade to Apache Druid 0.14.0-incubating URL: https://github.com/apache/calcite/pull/1174#discussion_r277683779 ## File path: druid/src/test/java/org/apache/calcite/test/DruidAdapterIT.java ## @@ -3

[GitHub] [calcite] nishantmonu51 commented on a change in pull request #1174: [CALCITE-3001] Upgrade to Apache Druid 0.14.0-incubating

2019-04-23 Thread GitBox
nishantmonu51 commented on a change in pull request #1174: [CALCITE-3001] Upgrade to Apache Druid 0.14.0-incubating URL: https://github.com/apache/calcite/pull/1174#discussion_r277683706 ## File path: druid/src/test/java/org/apache/calcite/test/DruidAdapterIT.java ## @@ -3

[GitHub] [calcite] chunweilei commented on a change in pull request #1164: [CALCITE-2998] RexCopier should support all rex types (Chunwei Lei)

2019-04-23 Thread GitBox
chunweilei commented on a change in pull request #1164: [CALCITE-2998] RexCopier should support all rex types (Chunwei Lei) URL: https://github.com/apache/calcite/pull/1164#discussion_r277601839 ## File path: core/src/test/java/org/apache/calcite/rex/RexBuilderTest.java ##

[GitHub] [calcite] chunweilei commented on a change in pull request #1164: [CALCITE-2998] RexCopier should support all rex types (Chunwei Lei)

2019-04-23 Thread GitBox
chunweilei commented on a change in pull request #1164: [CALCITE-2998] RexCopier should support all rex types (Chunwei Lei) URL: https://github.com/apache/calcite/pull/1164#discussion_r277600715 ## File path: core/src/main/java/org/apache/calcite/rex/RexCopier.java ## @@ -

[GitHub] [calcite] zabetak commented on a change in pull request #1164: [CALCITE-2998] RexCopier should support all rex types (Chunwei Lei)

2019-04-23 Thread GitBox
zabetak commented on a change in pull request #1164: [CALCITE-2998] RexCopier should support all rex types (Chunwei Lei) URL: https://github.com/apache/calcite/pull/1164#discussion_r277596536 ## File path: core/src/test/java/org/apache/calcite/rex/RexBuilderTest.java ## @@

[GitHub] [calcite] zabetak commented on a change in pull request #1164: [CALCITE-2998] RexCopier should support all rex types (Chunwei Lei)

2019-04-23 Thread GitBox
zabetak commented on a change in pull request #1164: [CALCITE-2998] RexCopier should support all rex types (Chunwei Lei) URL: https://github.com/apache/calcite/pull/1164#discussion_r277595197 ## File path: core/src/main/java/org/apache/calcite/rex/RexCopier.java ## @@ -52,

[GitHub] [calcite] chunweilei commented on issue #1177: [CALCITE-2453] Parsing list of SQL statements separated with a semicolon

2019-04-23 Thread GitBox
chunweilei commented on issue #1177: [CALCITE-2453] Parsing list of SQL statements separated with a semicolon URL: https://github.com/apache/calcite/pull/1177#issuecomment-485673611 PR is updated including: * Add some javadoc and merge some tests in SqlParserTest. * Use assertThat ra

[GitHub] [calcite] chunweilei commented on a change in pull request #1164: [CALCITE-2998] RexCopier should support all rex types (Chunwei Lei)

2019-04-23 Thread GitBox
chunweilei commented on a change in pull request #1164: [CALCITE-2998] RexCopier should support all rex types (Chunwei Lei) URL: https://github.com/apache/calcite/pull/1164#discussion_r277542067 ## File path: core/src/main/java/org/apache/calcite/rex/RexCopier.java ## @@ -