[GitHub] [calcite] DonnyZone opened a new pull request #1216: [CALCITE-3072] Generate right SQL for FLOOR functions in SparkSqlDialect

DonnyZone opened a new pull request #1216: [CALCITE-3072] Generate right SQL for FLOOR functions in SparkSqlDialect URL: https://github.com/apache/calcite/pull/1216 Rewrite `FLOOR` and `SUBSTRING` functions to generate SparkSql-specific query. `FLOOR->DATE_TRUNC` `SUBSTRING('Hello

[GitHub] [calcite] ambition119 commented on a change in pull request #1209: [CALCITE-2601] Support REVERSE(str) in SqlFunctions

ambition119 commented on a change in pull request #1209: [CALCITE-2601] Support REVERSE(str) in SqlFunctions URL: https://github.com/apache/calcite/pull/1209#discussion_r284536639 ## File path: core/src/main/java/org/apache/calcite/sql/fun/SqlStdOperatorTable.java ## @@

[GitHub] [calcite] chunweilei edited a comment on issue #1140: [CALCITE-2965] Implement string functions: REPEAT, SPACE, SOUNDEX, DI…

chunweilei edited a comment on issue #1140: [CALCITE-2965] Implement string functions: REPEAT, SPACE, SOUNDEX, DI… URL: https://github.com/apache/calcite/pull/1140#issuecomment-492904833 Rebase the PR to resolve conflicts.

[GitHub] [calcite] laurentgo commented on a change in pull request #1207: [CALCITE-3062] Do not populate provenanceMap if not debug

laurentgo commented on a change in pull request #1207: [CALCITE-3062] Do not populate provenanceMap if not debug URL: https://github.com/apache/calcite/pull/1207#discussion_r284529243 ## File path: core/src/main/java/org/apache/calcite/util/BlackholeMap.java ## @@ -0,0

[GitHub] [calcite] chunweilei commented on issue #1140: [CALCITE-2965] Implement string functions: REPEAT, SPACE, SOUNDEX, DI…

chunweilei commented on issue #1140: [CALCITE-2965] Implement string functions: REPEAT, SPACE, SOUNDEX, DI… URL: https://github.com/apache/calcite/pull/1140#issuecomment-492904833 Rebase the PR to resolve confilcts. This is

[GitHub] [calcite] ambition119 commented on a change in pull request #1209: [CALCITE-2601] Support REVERSE(str) in SqlFunctions

ambition119 commented on a change in pull request #1209: [CALCITE-2601] Support REVERSE(str) in SqlFunctions URL: https://github.com/apache/calcite/pull/1209#discussion_r284528796 ## File path: core/src/main/java/org/apache/calcite/sql/fun/SqlStdOperatorTable.java ## @@

[GitHub] [calcite] ambition119 commented on a change in pull request #1209: [CALCITE-2601] Support REVERSE(str) in SqlFunctions

ambition119 commented on a change in pull request #1209: [CALCITE-2601] Support REVERSE(str) in SqlFunctions URL: https://github.com/apache/calcite/pull/1209#discussion_r284528703 ## File path: site/_docs/reference.md ## @@ -2108,6 +2109,7 @@ semantics. | o |

[calcite-site] branch master updated: Publish website for Avatica Go 4.0.0 release

This is an automated email from the ASF dual-hosted git repository. francischuang 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 f0a5a10 Publish website for

[calcite] branch master updated: [CALCITE-2282] Remove sql operator table from parser

This is an automated email from the ASF dual-hosted git repository. danny0405 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 051058b [CALCITE-2282] Remove sql operator

[GitHub] [calcite] danny0405 merged pull request #1205: [CALCITE-2282] Remove sql operator table from parser

danny0405 merged pull request #1205: [CALCITE-2282] Remove sql operator table from parser URL: https://github.com/apache/calcite/pull/1205 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [calcite] danny0405 commented on a change in pull request #1209: [CALCITE-2601] Support REVERSE(str) in SqlFunctions

danny0405 commented on a change in pull request #1209: [CALCITE-2601] Support REVERSE(str) in SqlFunctions URL: https://github.com/apache/calcite/pull/1209#discussion_r284517360 ## File path: site/_docs/reference.md ## @@ -2108,6 +2109,7 @@ semantics. | o | GREATEST(expr

[GitHub] [calcite] danny0405 commented on a change in pull request #1209: [CALCITE-2601] Support REVERSE(str) in SqlFunctions

danny0405 commented on a change in pull request #1209: [CALCITE-2601] Support REVERSE(str) in SqlFunctions URL: https://github.com/apache/calcite/pull/1209#discussion_r284517159 ## File path: core/src/main/java/org/apache/calcite/sql/fun/SqlStdOperatorTable.java ## @@

[GitHub] [calcite] danny0405 commented on a change in pull request #1207: [CALCITE-3062] Do not populate provenanceMap if not debug

danny0405 commented on a change in pull request #1207: [CALCITE-3062] Do not populate provenanceMap if not debug URL: https://github.com/apache/calcite/pull/1207#discussion_r284515931 ## File path: core/src/main/java/org/apache/calcite/util/BlackholeMap.java ## @@ -0,0

[GitHub] [calcite] ambition119 commented on issue #1209: [CALCITE-2601] Support REVERSE(str) in SqlFunctions

ambition119 commented on issue #1209: [CALCITE-2601] Support REVERSE(str) in SqlFunctions URL: https://github.com/apache/calcite/pull/1209#issuecomment-492883862 @chunweilei @julianhyde I fix the issue comment code, and commit build success. But PR 1 failing check, so I can comment

[GitHub] [calcite] ambition119 commented on a change in pull request #1209: [CALCITE-2601] Support REVERSE(str) in SqlFunctions

ambition119 commented on a change in pull request #1209: [CALCITE-2601] Support REVERSE(str) in SqlFunctions URL: https://github.com/apache/calcite/pull/1209#discussion_r284282023 ## File path: core/src/test/java/org/apache/calcite/sql/test/DocumentationTest.java ## @@

[GitHub] [calcite] jcamachor commented on issue #1213: [CALCITE-3066] RelToSqlConverter may incorrectly throw an AssertionEr…

jcamachor commented on issue #1213: [CALCITE-3066] RelToSqlConverter may incorrectly throw an AssertionEr… URL: https://github.com/apache/calcite/pull/1213#issuecomment-492765822 Thanks @wangzzu , @rubenada , I've merged this change.

[calcite] 01/02: [CALCITE-3066] RelToSqlConverter may incorrectly throw an AssertionError for some decimal literals

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 commit d6896202c865b38b7821f1e0b644e1de0c95eda2 Author: Jesus Camacho Rodriguez AuthorDate: Mon May 13 18:35:09 2019 -0700

[GitHub] [calcite] asfgit closed pull request #1213: [CALCITE-3066] RelToSqlConverter may incorrectly throw an AssertionEr…

asfgit closed pull request #1213: [CALCITE-3066] RelToSqlConverter may incorrectly throw an AssertionEr… URL: https://github.com/apache/calcite/pull/1213 This is an automated message from the Apache Git Service. To respond

[calcite] 02/02: [CALCITE-3052] Error while applying rule MaterializedViewAggregateRule(Project-Aggregate): ArrayIndexOutOfBoundsException

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 commit 046bb81a9a4ac3e5b63c0ca4a0141446df325a44 Author: Jesus Camacho Rodriguez AuthorDate: Mon May 13 10:34:47 2019 -0700

[GitHub] [calcite] asfgit closed pull request #1212: [CALCITE-3052] Error while applying rule MaterializedViewAggregateRul…

asfgit closed pull request #1212: [CALCITE-3052] Error while applying rule MaterializedViewAggregateRul… URL: https://github.com/apache/calcite/pull/1212 This is an automated message from the Apache Git Service. To respond

[GitHub] [calcite] hsyuan merged pull request #1214: [CALCITE-3028] Support FULL OUTER JOIN with AggregateJoinTransposeRul…

hsyuan merged pull request #1214: [CALCITE-3028] Support FULL OUTER JOIN with AggregateJoinTransposeRul… URL: https://github.com/apache/calcite/pull/1214 This is an automated message from the Apache Git Service. To respond

[calcite] branch master updated: [CALCITE-3028] Support FULL OUTER JOIN with AggregateJoinTransposeRule (Vineet Garg)

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 a5b382a [CALCITE-3028] Support FULL OUTER JOIN

[GitHub] [calcite] hsyuan commented on issue #1207: [CALCITE-3062] Do not populate provenanceMap if not debug

hsyuan commented on issue #1207: [CALCITE-3062] Do not populate provenanceMap if not debug URL: https://github.com/apache/calcite/pull/1207#issuecomment-492748946 Thanks for the explanation. Looks good to me. This is an

[GitHub] [calcite] XuMingmin commented on a change in pull request #1127: [CALCITE-2913] Adapter for Apache Kafka

XuMingmin commented on a change in pull request #1127: [CALCITE-2913] Adapter for Apache Kafka URL: https://github.com/apache/calcite/pull/1127#discussion_r284359183 ## File path: kafka/src/main/java/org/apache/calcite/adapter/kafka/KafkaMessageEnumerator.java ## @@ -0,0

[GitHub] [calcite] XuMingmin commented on a change in pull request #1127: [CALCITE-2913] Adapter for Apache Kafka

XuMingmin commented on a change in pull request #1127: [CALCITE-2913] Adapter for Apache Kafka URL: https://github.com/apache/calcite/pull/1127#discussion_r284358866 ## File path: site/_docs/kafka_adapter.md ## @@ -0,0 +1,103 @@ +--- +layout: docs +title: Kafka adapter

[GitHub] [calcite] laurentgo commented on a change in pull request #1207: [CALCITE-3062] Do not populate provenanceMap if not debug

laurentgo commented on a change in pull request #1207: [CALCITE-3062] Do not populate provenanceMap if not debug URL: https://github.com/apache/calcite/pull/1207#discussion_r284352662 ## File path: core/src/main/java/org/apache/calcite/plan/volcano/VolcanoPlanner.java ##

[GitHub] [calcite] asereda-gs commented on a change in pull request #1180: [CALCITE-3023] Upgrade to Elasticsearch 7.0.0

asereda-gs commented on a change in pull request #1180: [CALCITE-3023] Upgrade to Elasticsearch 7.0.0 URL: https://github.com/apache/calcite/pull/1180#discussion_r284290461 ## File path: pom.xml ## @@ -75,7 +75,7 @@ limitations under the License. 2.5.0 3.8

[GitHub] [calcite] asereda-gs commented on issue #1180: [CALCITE-3023] Upgrade to Elasticsearch 7.0.0

asereda-gs commented on issue #1180: [CALCITE-3023] Upgrade to Elasticsearch 7.0.0 URL: https://github.com/apache/calcite/pull/1180#issuecomment-492679782 Can you also change documentation (`site`) to be more explicit that elastic search types are not supported (only indexes) ? We may

[GitHub] [calcite] XuQianJin-Stars edited a comment on issue #1168: [CALCITE-3005] Implement string functions: LEFT, RIGHT

XuQianJin-Stars edited a comment on issue #1168: [CALCITE-3005] Implement string functions: LEFT, RIGHT URL: https://github.com/apache/calcite/pull/1168#issuecomment-492517154 well，I will do it later. This is an automated

[calcite-avatica-go] 01/01: [CALCITE-3026] Add news item and set date for 4.0.0 release

This is an automated email from the ASF dual-hosted git repository. francischuang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite-avatica-go.git commit ca36f5430b3c81d9fbb21b0b65b3334aaf6eeb2f Author: Francis Chuang AuthorDate: Wed May 15 22:19:55 2019

[calcite-avatica-go] branch master updated (7b197cb -> ca36f54)

This is an automated email from the ASF dual-hosted git repository. francischuang pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/calcite-avatica-go.git. discard 7b197cb [CALCITE-3026] Add news item for 4.0.0 release new ca36f54 [CALCITE-3026] Add news

[GitHub] [calcite] danny0405 commented on a change in pull request #1127: [CALCITE-2913] Adapter for Apache Kafka

danny0405 commented on a change in pull request #1127: [CALCITE-2913] Adapter for Apache Kafka URL: https://github.com/apache/calcite/pull/1127#discussion_r284226267 ## File path: kafka/src/main/java/org/apache/calcite/adapter/kafka/KafkaMessageEnumerator.java ## @@ -0,0

[calcite-avatica-go] branch master updated: [CALCITE-3026] Add news item for 4.0.0 release

This is an automated email from the ASF dual-hosted git repository. francischuang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/calcite-avatica-go.git The following commit(s) were added to refs/heads/master by this push: new 7b197cb [CALCITE-3026] Add

svn commit: r34050 - in /release/calcite: apache-calcite-avatica-go-3.1.0/ apache-calcite-avatica-go-4.0.0/

Author: francischuang Date: Wed May 15 12:08:25 2019 New Revision: 34050 Log: Release apache-calcite-avatica-go-4.0.0 Added: release/calcite/apache-calcite-avatica-go-4.0.0/ release/calcite/apache-calcite-avatica-go-4.0.0/apache-calcite-avatica-go-4.0.0-src.tar.gz (with props)

[calcite-avatica-go] tag v4.0.0 created (now 3790ef5)

This is an automated email from the ASF dual-hosted git repository. francischuang pushed a change to tag v4.0.0 in repository https://gitbox.apache.org/repos/asf/calcite-avatica-go.git. at 3790ef5 (commit) No new revisions were added by this update.

[GitHub] [calcite] ambition119 commented on issue #1209: [CALCITE-2601] Support REVERSE(str) in SqlFunctions

ambition119 commented on issue #1209: [CALCITE-2601] Support REVERSE(str) in SqlFunctions URL: https://github.com/apache/calcite/pull/1209#issuecomment-492607664 > Please address julian's comments: > > > It's not a standard function. Can you create a new function category "j" (for

[GitHub] [calcite] chunweilei opened a new pull request #1215: [CALCITE-3068] testSubprogram() does not test whether subprogram gets re-executed

chunweilei opened a new pull request #1215: [CALCITE-3068] testSubprogram() does not test whether subprogram gets re-executed URL: https://github.com/apache/calcite/pull/1215 JIRA: https://issues.apache.org/jira/browse/CALCITE-3068

[GitHub] [calcite] KhawlaMhb commented on a change in pull request #1211: [CALCITE-2624] Add a rule to copy a sort below a join operator

KhawlaMhb commented on a change in pull request #1211: [CALCITE-2624] Add a rule to copy a sort below a join operator URL: https://github.com/apache/calcite/pull/1211#discussion_r284144380 ## File path: core/src/test/resources/org/apache/calcite/test/RelOptRulesTest.xml ##

[GitHub] [calcite] jacques-n commented on a change in pull request #1207: [CALCITE-3062] Do not populate provenanceMap if not debug

jacques-n commented on a change in pull request #1207: [CALCITE-3062] Do not populate provenanceMap if not debug URL: https://github.com/apache/calcite/pull/1207#discussion_r284141859 ## File path: core/src/main/java/org/apache/calcite/plan/volcano/VolcanoPlanner.java ##

[GitHub] [calcite] danny0405 commented on a change in pull request #1207: [CALCITE-3062] Do not populate provenanceMap if not debug

danny0405 commented on a change in pull request #1207: [CALCITE-3062] Do not populate provenanceMap if not debug URL: https://github.com/apache/calcite/pull/1207#discussion_r284129012 ## File path: core/src/main/java/org/apache/calcite/plan/volcano/VolcanoPlanner.java ##

[GitHub] [calcite] chunweilei commented on issue #1209: [CALCITE-2601] Support REVERSE(str) in SqlFunctions

chunweilei commented on issue #1209: [CALCITE-2601] Support REVERSE(str) in SqlFunctions URL: https://github.com/apache/calcite/pull/1209#issuecomment-492535196 Please address julian's comments: >It's not a standard function. Can you create a new function category "j" (for JDBC).

[GitHub] [calcite] rubenada commented on a change in pull request #1211: [CALCITE-2624] Add a rule to copy a sort below a join operator

rubenada commented on a change in pull request #1211: [CALCITE-2624] Add a rule to copy a sort below a join operator URL: https://github.com/apache/calcite/pull/1211#discussion_r284110263 ## File path: core/src/test/resources/org/apache/calcite/test/RelOptRulesTest.xml ##

[GitHub] [calcite] jcamachor commented on a change in pull request #1212: [CALCITE-3052] Error while applying rule MaterializedViewAggregateRul…

jcamachor commented on a change in pull request #1212: [CALCITE-3052] Error while applying rule MaterializedViewAggregateRul… URL: https://github.com/apache/calcite/pull/1212#discussion_r284100203 ## File path:

[GitHub] [calcite] XuQianJin-Stars commented on issue #1168: [CALCITE-3005] Implement string functions: LEFT, RIGHT

XuQianJin-Stars commented on issue #1168: [CALCITE-3005] Implement string functions: LEFT, RIGHT URL: https://github.com/apache/calcite/pull/1168#issuecomment-492517154 well，I will do it later. | | 徐前进 | | 邮箱：x1q...@163.com | 签名由 网易邮箱大师 定制 On

[GitHub] [calcite] chunweilei commented on a change in pull request #1212: [CALCITE-3052] Error while applying rule MaterializedViewAggregateRul…

chunweilei commented on a change in pull request #1212: [CALCITE-3052] Error while applying rule MaterializedViewAggregateRul… URL: https://github.com/apache/calcite/pull/1212#discussion_r284097882 ## File path:

[GitHub] [calcite] chunweilei commented on issue #1168: [CALCITE-3005] Implement string functions: LEFT, RIGHT

chunweilei commented on issue #1168: [CALCITE-3005] Implement string functions: LEFT, RIGHT URL: https://github.com/apache/calcite/pull/1168#issuecomment-492516711 @XuQianJin-Stars could you please rebase it and move it to `SqlLibraryOperators` since it is not a standard function?