[GitHub] [calcite] vaibhavjain-dm opened a new pull request #1622: [CALCITE-3556] : Handling of TO_NUMBER method

vaibhavjain-dm opened a new pull request #1622: [CALCITE-3556] : Handling of TO_NUMBER method URL: https://github.com/apache/calcite/pull/1622 [CALCITE-3556] : Handling of TO_NUMBER method This is an automated message from

[GitHub] [calcite] danny0405 commented on issue #1133: [CALCITE-2208] MaterializedViewTable use connection config to parse s…

danny0405 commented on issue #1133: [CALCITE-2208] MaterializedViewTable use connection config to parse s… URL: https://github.com/apache/calcite/pull/1133#issuecomment-562005320 Close this PR because of lack of time, if anyone is interested in this topic, discuss it in the JIRA issue.

[GitHub] [calcite] danny0405 closed pull request #1133: [CALCITE-2208] MaterializedViewTable use connection config to parse s…

danny0405 closed pull request #1133: [CALCITE-2208] MaterializedViewTable use connection config to parse s… URL: https://github.com/apache/calcite/pull/1133 This is an automated message from the Apache Git Service. To

[GitHub] [calcite] danny0405 closed pull request #1417: [CALCITE-3244] RelDecorrelator unable to decorrelate expression with …

danny0405 closed pull request #1417: [CALCITE-3244] RelDecorrelator unable to decorrelate expression with … URL: https://github.com/apache/calcite/pull/1417 This is an automated message from the Apache Git Service. To

[GitHub] [calcite] danny0405 commented on issue #1417: [CALCITE-3244] RelDecorrelator unable to decorrelate expression with …

danny0405 commented on issue #1417: [CALCITE-3244] RelDecorrelator unable to decorrelate expression with … URL: https://github.com/apache/calcite/pull/1417#issuecomment-562004910 Close this PR because i have no idea how to fix it probably now.

[GitHub] [calcite] asereda-gs commented on issue #1399: [CALCITE-3281] Support mixed Primitive types for BinaryExpression evaluate method.

asereda-gs commented on issue #1399: [CALCITE-3281] Support mixed Primitive types for BinaryExpression evaluate method. URL: https://github.com/apache/calcite/pull/1399#issuecomment-561994904 Thanks for PR, @yanlin-Lynn !

[calcite] branch master updated (f1e51e6 -> 67ed322)

This is an automated email from the ASF dual-hosted git repository. sereda pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git. from f1e51e6 [CALCITE-3561] Support using unnest in Interpreter (Wang Yanlin) add 67ed322 [CALCITE-3281]

[GitHub] [calcite] asereda-gs merged pull request #1399: [CALCITE-3281] Support mixed Primitive types for BinaryExpression evaluate method.

asereda-gs merged pull request #1399: [CALCITE-3281] Support mixed Primitive types for BinaryExpression evaluate method. URL: https://github.com/apache/calcite/pull/1399 This is an automated message from the Apache Git

[GitHub] [calcite] LastSmile732 commented on a change in pull request #1624: [Calcite 3558] Fixing indexing variable for the column reordering loop

LastSmile732 commented on a change in pull request #1624: [Calcite 3558] Fixing indexing variable for the column reordering loop URL: https://github.com/apache/calcite/pull/1624#discussion_r354124118 ## File path:

[GitHub] [calcite] amaliujia commented on issue #1596: Add guidance of how to run gradle command(e.g. check) in a

amaliujia commented on issue #1596: Add guidance of how to run gradle command(e.g. check) in a URL: https://github.com/apache/calcite/pull/1596#issuecomment-561983644 @asereda-gs done This is an automated message from the

[GitHub] [calcite] yanlin-Lynn commented on issue #1626: [CALCITE-3561] Support using unnest in Interpreter

yanlin-Lynn commented on issue #1626: [CALCITE-3561] Support using unnest in Interpreter URL: https://github.com/apache/calcite/pull/1626#issuecomment-561980030 Thanks for review and merge, I'll delete the branch. This is an

[GitHub] [calcite] jinxing64 edited a comment on issue #1635: [CALCITE-3570] Support MutableSnapshot in MutableRel

jinxing64 edited a comment on issue #1635: [CALCITE-3570] Support MutableSnapshot in MutableRel URL: https://github.com/apache/calcite/pull/1635#issuecomment-561975389 Can you explain in what scenario `MutableSnapshot` is needed ? Since it's not supported in SubstitutionVisitor at this

[GitHub] [calcite] jinxing64 commented on issue #1635: [CALCITE-3570] Support MutableSnapshot in MutableRel

jinxing64 commented on issue #1635: [CALCITE-3570] Support MutableSnapshot in MutableRel URL: https://github.com/apache/calcite/pull/1635#issuecomment-561975389 Can you explain in what scenario `MutableSnapshot` is needed ?

[GitHub] [calcite] hsyuan commented on issue #1573: [CALCITE-3496] Hive dialect and MS SQL dialect support with cube and with rollup

hsyuan commented on issue #1573: [CALCITE-3496] Hive dialect and MS SQL dialect support with cube and with rollup URL: https://github.com/apache/calcite/pull/1573#issuecomment-561966309 Yeah, I commented in the JIRA: https://issues.apache.org/jira/browse/CALCITE-3496

[GitHub] [calcite] xy2953396112 commented on issue #1573: [CALCITE-3496] Hive dialect and MS SQL dialect support with cube and with rollup

xy2953396112 commented on issue #1573: [CALCITE-3496] Hive dialect and MS SQL dialect support with cube and with rollup URL: https://github.com/apache/calcite/pull/1573#issuecomment-561966045 @hsyuan is there some problems in this pr?

[GitHub] [calcite] dependabot[bot] commented on issue #1580: Bump jetty.version from 9.4.11.v20180605 to 9.4.22.v20191022

dependabot[bot] commented on issue #1580: Bump jetty.version from 9.4.11.v20180605 to 9.4.22.v20191022 URL: https://github.com/apache/calcite/pull/1580#issuecomment-561964540 OK, I won't notify you again about this release, but will get in touch when a new version is available. If

[GitHub] [calcite] asereda-gs closed pull request #1580: Bump jetty.version from 9.4.11.v20180605 to 9.4.22.v20191022

asereda-gs closed pull request #1580: Bump jetty.version from 9.4.11.v20180605 to 9.4.22.v20191022 URL: https://github.com/apache/calcite/pull/1580 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [calcite] asereda-gs commented on issue #1580: Bump jetty.version from 9.4.11.v20180605 to 9.4.22.v20191022

asereda-gs commented on issue #1580: Bump jetty.version from 9.4.11.v20180605 to 9.4.22.v20191022 URL: https://github.com/apache/calcite/pull/1580#issuecomment-561964345 Not sure how Dependabot works with gradle This is an

[GitHub] [calcite] asereda-gs commented on issue #1626: [CALCITE-3561] Support using unnest in Interpreter

asereda-gs commented on issue #1626: [CALCITE-3561] Support using unnest in Interpreter URL: https://github.com/apache/calcite/pull/1626#issuecomment-561963699 Thank you for PR, @yanlin-Lynn ! This is an automated message

[GitHub] [calcite] asereda-gs merged pull request #1626: [CALCITE-3561] Support using unnest in Interpreter

asereda-gs merged pull request #1626: [CALCITE-3561] Support using unnest in Interpreter URL: https://github.com/apache/calcite/pull/1626 This is an automated message from the Apache Git Service. To respond to the message,

[calcite] branch master updated (0aaf4f6 -> f1e51e6)

This is an automated email from the ASF dual-hosted git repository. sereda pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git. from 0aaf4f6 [CALCITE-3566] EnumerableIntersect and EnumerableMinus convert to Logical (xzh_dz) add f1e51e6

[GitHub] [calcite] chunweilei commented on issue #1634: [CALCITE-3246] NullPointerException while deserializing udf operator

chunweilei commented on issue #1634: [CALCITE-3246] NullPointerException while deserializing udf operator URL: https://github.com/apache/calcite/pull/1634#issuecomment-561960567 LGTM This is an automated message from the

[GitHub] [calcite] xy2953396112 opened a new pull request #1635: [CALCITE-3570] Support MutableSnapshot in MutableRel

xy2953396112 opened a new pull request #1635: [CALCITE-3570] Support MutableSnapshot in MutableRel URL: https://github.com/apache/calcite/pull/1635 Support MutableSnapshot in MutableRel.This PR implements mutual conversion between MutableSnapshot and LogicalSnapshot.

[GitHub] [calcite] yanlin-Lynn commented on a change in pull request #1626: [CALCITE-3561] Support using unnest in Interpreter

yanlin-Lynn commented on a change in pull request #1626: [CALCITE-3561] Support using unnest in Interpreter URL: https://github.com/apache/calcite/pull/1626#discussion_r354095900 ## File path: core/src/main/java/org/apache/calcite/interpreter/UncollectNode.java ## @@ -0,0

[GitHub] [calcite] asereda-gs commented on a change in pull request #1626: [CALCITE-3561] Support using unnest in Interpreter

asereda-gs commented on a change in pull request #1626: [CALCITE-3561] Support using unnest in Interpreter URL: https://github.com/apache/calcite/pull/1626#discussion_r354093122 ## File path: core/src/main/java/org/apache/calcite/interpreter/UncollectNode.java ## @@ -0,0

[GitHub] [calcite] asereda-gs commented on issue #1626: [CALCITE-3561] Support using unnest in Interpreter

asereda-gs commented on issue #1626: [CALCITE-3561] Support using unnest in Interpreter URL: https://github.com/apache/calcite/pull/1626#issuecomment-561947456 als pls squash commits into one This is an automated message

[GitHub] [calcite] danny0405 commented on issue #1521: [CALCITE-3429] AssertionError for user-defined table function with map argument

danny0405 commented on issue #1521: [CALCITE-3429] AssertionError for user-defined table function with map argument URL: https://github.com/apache/calcite/pull/1521#issuecomment-561944567 Sorry for the delay, reviewing now ~

[GitHub] [calcite] yanlin-Lynn opened a new pull request #1634: [CALCITE-3246] NullPointerException while deserializing udf operator

yanlin-Lynn opened a new pull request #1634: [CALCITE-3246] NullPointerException while deserializing udf operator URL: https://github.com/apache/calcite/pull/1634 The class of udf was lost in the pr [CALCITE-3177](https://github.com/apache/calcite/pull/1305), causing an

[GitHub] [calcite] yanlin-Lynn commented on issue #1478: [CALCITE-3378] AssertionError for checking RexNode implify

yanlin-Lynn commented on issue #1478: [CALCITE-3378] AssertionError for checking RexNode implify URL: https://github.com/apache/calcite/pull/1478#issuecomment-561940557 Resolve conflicts and force push. This is an automated

[GitHub] [calcite] yanlin-Lynn removed a comment on issue #1521: [CALCITE-3429] AssertionError for user-defined table function with map argument

yanlin-Lynn removed a comment on issue #1521: [CALCITE-3429] AssertionError for user-defined table function with map argument URL: https://github.com/apache/calcite/pull/1521#issuecomment-561940361 Resolve conflicts and force push.

[GitHub] [calcite] yanlin-Lynn commented on issue #1521: [CALCITE-3429] AssertionError for user-defined table function with map argument

yanlin-Lynn commented on issue #1521: [CALCITE-3429] AssertionError for user-defined table function with map argument URL: https://github.com/apache/calcite/pull/1521#issuecomment-561940361 Resolve conflicts and force push.

[GitHub] [calcite] yanlin-Lynn commented on a change in pull request #1626: [CALCITE-3561] Support using unnest in Interpreter

yanlin-Lynn commented on a change in pull request #1626: [CALCITE-3561] Support using unnest in Interpreter URL: https://github.com/apache/calcite/pull/1626#discussion_r354078727 ## File path: core/src/main/java/org/apache/calcite/interpreter/UncollectNode.java ## @@ -0,0

[GitHub] [calcite] yanlin-Lynn commented on issue #1630: [CALCITE-3567] Unnest support Map wrapped with RecordType

yanlin-Lynn commented on issue #1630: [CALCITE-3567] Unnest support Map wrapped with RecordType URL: https://github.com/apache/calcite/pull/1630#issuecomment-561929989 Thanks for review and merge. This is an automated

[GitHub] [calcite] danny0405 commented on a change in pull request #1629: [CALCITE-3550] Make SqlTypeAssignmentRules conversion mapping pluggable

danny0405 commented on a change in pull request #1629: [CALCITE-3550] Make SqlTypeAssignmentRules conversion mapping pluggable URL: https://github.com/apache/calcite/pull/1629#discussion_r354067939 ## File path:

[calcite] branch master updated (e2f407f -> 0aaf4f6)

This is an automated email from the ASF dual-hosted git repository. sereda pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git. omit e2f407f EnumerableIntersect and EnumerableMinus convert to Logical add 0aaf4f6 [CALCITE-3566]

[GitHub] [calcite] asereda-gs commented on issue #1596: Add guidance of how to run gradle command(e.g. check) in a

asereda-gs commented on issue #1596: Add guidance of how to run gradle command(e.g. check) in a URL: https://github.com/apache/calcite/pull/1596#issuecomment-561911747 Pls squash commits and add your full name in parentheses `(FirstName LastName)`

[GitHub] [calcite] xy2953396112 commented on issue #1627: [CALCITE-3566] Support EnumerableIntersect and EnumerableMinus convert to Logical

xy2953396112 commented on issue #1627: [CALCITE-3566] Support EnumerableIntersect and EnumerableMinus convert to Logical URL: https://github.com/apache/calcite/pull/1627#issuecomment-561910928 > Thanks, @xy2953396112, for this PR! @asereda-gs @chunweilei Thanks for you review.

[calcite] branch master updated (560d895 -> e2f407f)

This is an automated email from the ASF dual-hosted git repository. sereda pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git. from 560d895 [CALCITE-3567] Unnest support Map wrapped with RecordType (Wang Yanlin) add e2f407f

[GitHub] [calcite] asereda-gs merged pull request #1627: [CALCITE-3566] Support EnumerableIntersect and EnumerableMinus convert to Logical

asereda-gs merged pull request #1627: [CALCITE-3566] Support EnumerableIntersect and EnumerableMinus convert to Logical URL: https://github.com/apache/calcite/pull/1627 This is an automated message from the Apache Git

[GitHub] [calcite] asereda-gs commented on issue #1627: [CALCITE-3566] Support EnumerableIntersect and EnumerableMinus convert to Logical

asereda-gs commented on issue #1627: [CALCITE-3566] Support EnumerableIntersect and EnumerableMinus convert to Logical URL: https://github.com/apache/calcite/pull/1627#issuecomment-561910793 Thanks, @xy2953396112, for this PR!

[GitHub] [calcite] asereda-gs commented on issue #1630: [CALCITE-3567] Unnest support Map wrapped with RecordType

asereda-gs commented on issue #1630: [CALCITE-3567] Unnest support Map wrapped with RecordType URL: https://github.com/apache/calcite/pull/1630#issuecomment-561909276 Thanks for PR @yanlin-Lynn ! This is an automated message

[calcite] branch master updated (11d7916 -> 560d895)

This is an automated email from the ASF dual-hosted git repository. sereda pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git. from 11d7916 [CALCITE-3569] IndexOutOfBoundsException when pushing simplified filter to view add 560d895

[GitHub] [calcite] asereda-gs merged pull request #1630: [CALCITE-3567] Unnest support Map wrapped with RecordType

asereda-gs merged pull request #1630: [CALCITE-3567] Unnest support Map wrapped with RecordType URL: https://github.com/apache/calcite/pull/1630 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [calcite] asereda-gs commented on a change in pull request #1626: [CALCITE-3561] Support using unnest in Interpreter

asereda-gs commented on a change in pull request #1626: [CALCITE-3561] Support using unnest in Interpreter URL: https://github.com/apache/calcite/pull/1626#discussion_r354053829 ## File path: core/src/main/java/org/apache/calcite/interpreter/UncollectNode.java ## @@ -0,0

[calcite] branch master updated (a0931b7 -> 11d7916)

This is an automated email from the ASF dual-hosted git repository. sereda pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git. from a0931b7 [CALCITE-3285] EnumerableMergeJoin should support non-equi join condions (Jin Xing) add 11d7916

[GitHub] [calcite] asereda-gs merged pull request #1631: [CALCITE-3569] IndexOutOfBoundsException when pushing simplified filter to view

asereda-gs merged pull request #1631: [CALCITE-3569] IndexOutOfBoundsException when pushing simplified filter to view URL: https://github.com/apache/calcite/pull/1631 This is an automated message from the Apache Git

[GitHub] [calcite] asereda-gs commented on issue #1631: [CALCITE-3569] IndexOutOfBoundsException when pushing simplified filter to view

asereda-gs commented on issue #1631: [CALCITE-3569] IndexOutOfBoundsException when pushing simplified filter to view URL: https://github.com/apache/calcite/pull/1631#issuecomment-561903860 Thanks for PR @chunweilei We'll be releasing 1.22 soon

[GitHub] [calcite] xndai commented on issue #1543: [CALCITE-3455] Redundant rule firing for both logical and physical nodes

xndai commented on issue #1543: [CALCITE-3455] Redundant rule firing for both logical and physical nodes URL: https://github.com/apache/calcite/pull/1543#issuecomment-561899807 Thanks @zabetak for the pointer. After reviewing the whole thread, I now understand why @vlsi believes this is

[GitHub] [calcite] asereda-gs commented on issue #1313: [CALCITE-3190] ElasticsearchJson throws Exception when visitMappingProperties

asereda-gs commented on issue #1313: [CALCITE-3190] ElasticsearchJson throws Exception when visitMappingProperties URL: https://github.com/apache/calcite/pull/1313#issuecomment-561899978 @durpersi we're approaching 1.22 release. Do you think you can address review comments before then

[GitHub] [calcite] zabetak commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter

zabetak commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter URL: https://github.com/apache/calcite/pull/1591#discussion_r354039300 ## File path: redis/src/test/java/org/apache/calcite/adapter/redis/RedisAdapterITCaseBase.java ## @@ -0,0

[GitHub] [calcite] zabetak commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter

zabetak commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter URL: https://github.com/apache/calcite/pull/1591#discussion_r354037791 ## File path: redis/src/main/java/org/apache/calcite/adapter/redis/package-info.java ## @@ -0,0 +1,27 @@

[GitHub] [calcite] zabetak commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter

zabetak commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter URL: https://github.com/apache/calcite/pull/1591#discussion_r354037014 ## File path: core/src/main/java/org/apache/calcite/config/CalciteSystemProperty.java ## @@ -203,6 +203,12

[GitHub] [calcite] asereda-gs commented on a change in pull request #1632: [CALCITE-3560] Additional calcite.util.Source implementation for generic text source (eg. CharSource)

asereda-gs commented on a change in pull request #1632: [CALCITE-3560] Additional calcite.util.Source implementation for generic text source (eg. CharSource) URL: https://github.com/apache/calcite/pull/1632#discussion_r354025708 ## File path:

[GitHub] [calcite] julianhyde commented on issue #1633: Fix copyright headers, move copyright owner from Resources.java to NOTICE

julianhyde commented on issue #1633: Fix copyright headers, move copyright owner from Resources.java to NOTICE URL: https://github.com/apache/calcite/pull/1633#issuecomment-561872817 The link you cite is "SOURCE FILE HEADERS FOR CODE DEVELOPED AT THE ASF", does not apply here.

[GitHub] [calcite] hsyuan closed pull request #1576: [CALCITE-3285] EnumerableMergeJoin should support non-equi join condi…

hsyuan closed pull request #1576: [CALCITE-3285] EnumerableMergeJoin should support non-equi join condi… URL: https://github.com/apache/calcite/pull/1576 This is an automated message from the Apache Git Service. To respond

[calcite] branch master updated (ecc68ac -> a0931b7)

This is an automated email from the ASF dual-hosted git repository. hyuan pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git. from ecc68ac [CALCITE-3548] unlock ./gradlew :ubenchmark:jmh to run benchmarks add a0931b7 [CALCITE-3285]

[GitHub] [calcite] vlsi commented on a change in pull request #1633: Fix copyright headers, move copyright owner from Resources.java to NOTICE

vlsi commented on a change in pull request #1633: Fix copyright headers, move copyright owner from Resources.java to NOTICE URL: https://github.com/apache/calcite/pull/1633#discussion_r354015817 ## File path: core/src/main/java/org/apache/calcite/runtime/Resources.java ##

[GitHub] [calcite] vlsi commented on a change in pull request #1633: Fix copyright headers, move copyright owner from Resources.java to NOTICE

vlsi commented on a change in pull request #1633: Fix copyright headers, move copyright owner from Resources.java to NOTICE URL: https://github.com/apache/calcite/pull/1633#discussion_r354015817 ## File path: core/src/main/java/org/apache/calcite/runtime/Resources.java ##

[GitHub] [calcite] vlsi opened a new pull request #1633: Fix copyright headers, move copyright owner from Resources.java to NOTICE

vlsi opened a new pull request #1633: Fix copyright headers, move copyright owner from Resources.java to NOTICE URL: https://github.com/apache/calcite/pull/1633 Copyright owner should be moved from header to NOTICE as per https://www.apache.org/legal/src-headers.html#headers

[GitHub] [calcite] julianhyde commented on a change in pull request #1624: [Calcite 3558] Fixing indexing variable for the column reordering loop

julianhyde commented on a change in pull request #1624: [Calcite 3558] Fixing indexing variable for the column reordering loop URL: https://github.com/apache/calcite/pull/1624#discussion_r353999204 ## File path:

[GitHub] [calcite] asereda-gs commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter

asereda-gs commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter URL: https://github.com/apache/calcite/pull/1591#discussion_r353990471 ## File path: redis/src/main/java/org/apache/calcite/adapter/redis/RedisJedisManager.java ## @@ -0,0

[GitHub] [calcite] asereda-gs commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter

asereda-gs commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter URL: https://github.com/apache/calcite/pull/1591#discussion_r353987894 ## File path: redis/src/main/java/org/apache/calcite/adapter/redis/RedisDataBase.java ## @@ -0,0 +1,145

[GitHub] [calcite] asereda-gs commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter

asereda-gs commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter URL: https://github.com/apache/calcite/pull/1591#discussion_r353995002 ## File path: redis/src/test/java/org/apache/calcite/adapter/redis/RedisAdapterITCaseBase.java ## @@

[GitHub] [calcite] asereda-gs commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter

asereda-gs commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter URL: https://github.com/apache/calcite/pull/1591#discussion_r353984537 ## File path: redis/src/test/java/org/apache/calcite/adapter/redis/RedisITCaseBase.java ## @@ -0,0 +1,77

[GitHub] [calcite] asereda-gs commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter

asereda-gs commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter URL: https://github.com/apache/calcite/pull/1591#discussion_r353988841 ## File path: redis/src/main/java/org/apache/calcite/adapter/redis/RedisDataBase.java ## @@ -0,0 +1,145

[GitHub] [calcite] asereda-gs commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter

asereda-gs commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter URL: https://github.com/apache/calcite/pull/1591#discussion_r353986557 ## File path: redis/src/main/java/org/apache/calcite/adapter/redis/RedisDataBase.java ## @@ -0,0 +1,145

[GitHub] [calcite] asereda-gs commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter

asereda-gs commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter URL: https://github.com/apache/calcite/pull/1591#discussion_r353992696 ## File path: redis/src/main/resources/log4j.properties ## @@ -0,0 +1,28 @@ +# Licensed to the Apache

[GitHub] [calcite] asereda-gs commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter

asereda-gs commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter URL: https://github.com/apache/calcite/pull/1591#discussion_r353990724 ## File path: redis/src/main/java/org/apache/calcite/adapter/redis/RedisJedisManager.java ## @@ -0,0

[GitHub] [calcite] asereda-gs commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter

asereda-gs commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter URL: https://github.com/apache/calcite/pull/1591#discussion_r353997327 ## File path: redis/src/main/java/org/apache/calcite/adapter/redis/RedisDataProcess.java ## @@ -0,0

[GitHub] [calcite] asereda-gs commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter

asereda-gs commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter URL: https://github.com/apache/calcite/pull/1591#discussion_r353989295 ## File path: redis/src/main/java/org/apache/calcite/adapter/redis/RedisEnumerator.java ## @@ -0,0 +1,89

[GitHub] [calcite] amaliujia commented on a change in pull request #1587: [CALCITE-3272] Support TUMBLE as Table Valued Function including an enumerable implementation, stream.iq and DESCRIPTOR

amaliujia commented on a change in pull request #1587: [CALCITE-3272] Support TUMBLE as Table Valued Function including an enumerable implementation, stream.iq and DESCRIPTOR URL: https://github.com/apache/calcite/pull/1587#discussion_r353987008 ## File path:

[GitHub] [calcite] amaliujia commented on a change in pull request #1587: [CALCITE-3272] Support TUMBLE as Table Valued Function including an enumerable implementation, stream.iq and DESCRIPTOR

amaliujia commented on a change in pull request #1587: [CALCITE-3272] Support TUMBLE as Table Valued Function including an enumerable implementation, stream.iq and DESCRIPTOR URL: https://github.com/apache/calcite/pull/1587#discussion_r353987008 ## File path:

[GitHub] [calcite] hsyuan commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter

hsyuan commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter URL: https://github.com/apache/calcite/pull/1591#discussion_r353977593 ## File path: redis/src/test/java/org/apache/calcite/adapter/redis/RedisITCaseBase.java ## @@ -0,0 +1,77 @@

[GitHub] [calcite] hsyuan commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter

hsyuan commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter URL: https://github.com/apache/calcite/pull/1591#discussion_r353976962 ## File path: redis/src/test/java/org/apache/calcite/adapter/redis/RedisServerMini.java ## @@ -0,0 +1,120 @@

[GitHub] [calcite] asereda-gs commented on a change in pull request #1632: [CALCITE-3560] Additional calcite.util.Source implementation for generic text source (eg. CharSource)

asereda-gs commented on a change in pull request #1632: [CALCITE-3560] Additional calcite.util.Source implementation for generic text source (eg. CharSource) URL: https://github.com/apache/calcite/pull/1632#discussion_r353971306 ## File path:

[GitHub] [calcite] hsyuan commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter

hsyuan commented on a change in pull request #1591: [CALCITE-3510] Implement Redis adapter URL: https://github.com/apache/calcite/pull/1591#discussion_r353970688 ## File path: redis/src/main/java/org/apache/calcite/adapter/redis/RedisDataType.java ## @@ -0,0 +1,91 @@

[GitHub] [calcite] vlsi commented on a change in pull request #1632: [CALCITE-3560] Additional calcite.util.Source implementation for generic text source (eg. CharSource)

vlsi commented on a change in pull request #1632: [CALCITE-3560] Additional calcite.util.Source implementation for generic text source (eg. CharSource) URL: https://github.com/apache/calcite/pull/1632#discussion_r353963017 ## File path:

[GitHub] [calcite] hsyuan commented on issue #1625: [CALCITE-3559] Drop HydromaticFileSetCheck, upgrade Checkstyle

hsyuan commented on issue #1625: [CALCITE-3559] Drop HydromaticFileSetCheck, upgrade Checkstyle URL: https://github.com/apache/calcite/pull/1625#issuecomment-561823031 LGTM This is an automated message from the Apache Git

[GitHub] [calcite] hsyuan commented on a change in pull request #1629: [CALCITE-3550] Make SqlTypeAssignmentRules conversion mapping pluggable

hsyuan commented on a change in pull request #1629: [CALCITE-3550] Make SqlTypeAssignmentRules conversion mapping pluggable URL: https://github.com/apache/calcite/pull/1629#discussion_r353958281 ## File path:

[GitHub] [calcite] vlsi commented on a change in pull request #1632: [CALCITE-3560] Additional calcite.util.Source implementation for generic text source (eg. CharSource)

vlsi commented on a change in pull request #1632: [CALCITE-3560] Additional calcite.util.Source implementation for generic text source (eg. CharSource) URL: https://github.com/apache/calcite/pull/1632#discussion_r353958184 ## File path:

[GitHub] [calcite] asereda-gs commented on a change in pull request #1632: [CALCITE-3560] Additional calcite.util.Source implementation for generic text source (eg. CharSource)

asereda-gs commented on a change in pull request #1632: [CALCITE-3560] Additional calcite.util.Source implementation for generic text source (eg. CharSource) URL: https://github.com/apache/calcite/pull/1632#discussion_r353946698 ## File path:

[GitHub] [calcite] asereda-gs commented on a change in pull request #1632: [CALCITE-3560] Additional calcite.util.Source implementation for generic text source (eg. CharSource)

asereda-gs commented on a change in pull request #1632: [CALCITE-3560] Additional calcite.util.Source implementation for generic text source (eg. CharSource) URL: https://github.com/apache/calcite/pull/1632#discussion_r353946114 ## File path:

[GitHub] [calcite] vlsi commented on a change in pull request #1632: [CALCITE-3560] Additional calcite.util.Source implementation for generic text source (eg. CharSource)

vlsi commented on a change in pull request #1632: [CALCITE-3560] Additional calcite.util.Source implementation for generic text source (eg. CharSource) URL: https://github.com/apache/calcite/pull/1632#discussion_r353944747 ## File path:

[GitHub] [calcite] vlsi commented on a change in pull request #1632: [CALCITE-3560] Additional calcite.util.Source implementation for generic text source (eg. CharSource)

vlsi commented on a change in pull request #1632: [CALCITE-3560] Additional calcite.util.Source implementation for generic text source (eg. CharSource) URL: https://github.com/apache/calcite/pull/1632#discussion_r353943555 ## File path:

[GitHub] [calcite] asereda-gs opened a new pull request #1632: [CALCITE-3560] Additional calcite.util.Source implementation for generic text source (eg. CharSource)

asereda-gs opened a new pull request #1632: [CALCITE-3560] Additional calcite.util.Source implementation for generic text source (eg. CharSource) URL: https://github.com/apache/calcite/pull/1632 Currently calcite calcite.util.Source interface can be built only from File or URL. This

[GitHub] [calcite] julianhyde commented on a change in pull request #1610: [CALCITE-3525] RexSimplify: eliminate redundant rex calls in OR

julianhyde commented on a change in pull request #1610: [CALCITE-3525] RexSimplify: eliminate redundant rex calls in OR URL: https://github.com/apache/calcite/pull/1610#discussion_r353921915 ## File path:

[GitHub] [calcite] julianhyde commented on a change in pull request #1610: [CALCITE-3525] RexSimplify: eliminate redundant rex calls in OR

julianhyde commented on a change in pull request #1610: [CALCITE-3525] RexSimplify: eliminate redundant rex calls in OR URL: https://github.com/apache/calcite/pull/1610#discussion_r353919304 ## File path:

[GitHub] [calcite] julianhyde commented on issue #1547: [CALCITE-3460] Poor performance in RexReplacer for large queries

julianhyde commented on issue #1547: [CALCITE-3460] Poor performance in RexReplacer for large queries URL: https://github.com/apache/calcite/pull/1547#issuecomment-561779708 In case you missed my comment in the JIRA case, we must have a unit test for this change.

[GitHub] [calcite] chunweilei opened a new pull request #1631: [CALCITE-3569] IndexOutOfBoundsException when pushing simplified filter to view

chunweilei opened a new pull request #1631: [CALCITE-3569] IndexOutOfBoundsException when pushing simplified filter to view URL: https://github.com/apache/calcite/pull/1631 JIRA: https://issues.apache.org/jira/browse/CALCITE-3569.

[GitHub] [calcite] rubenada commented on a change in pull request #1613: [CALCITE-3542] Implement RepeatUnion All=false

rubenada commented on a change in pull request #1613: [CALCITE-3542] Implement RepeatUnion All=false URL: https://github.com/apache/calcite/pull/1613#discussion_r353771215 ## File path: core/src/test/java/org/apache/calcite/test/enumerable/EnumerableRepeatUnionHierarchyTest.java

[GitHub] [calcite] danny0405 commented on a change in pull request #1613: [CALCITE-3542] Implement RepeatUnion All=false

danny0405 commented on a change in pull request #1613: [CALCITE-3542] Implement RepeatUnion All=false URL: https://github.com/apache/calcite/pull/1613#discussion_r353757711 ## File path: core/src/test/java/org/apache/calcite/test/enumerable/EnumerableRepeatUnionHierarchyTest.java

[GitHub] [calcite] vlsi commented on issue #1547: [CALCITE-3460] Poor performance in RexReplacer for large queries

vlsi commented on issue #1547: [CALCITE-3460] Poor performance in RexReplacer for large queries URL: https://github.com/apache/calcite/pull/1547#issuecomment-561651656 LGTM This is an automated message from the Apache Git

[GitHub] [calcite] vlsi commented on a change in pull request #1547: [CALCITE-3460] Poor performance in RexReplacer for large queries

vlsi commented on a change in pull request #1547: [CALCITE-3460] Poor performance in RexReplacer for large queries URL: https://github.com/apache/calcite/pull/1547#discussion_r353749942 ## File path: core/src/main/java/org/apache/calcite/rel/rules/ReduceExpressionsRule.java

[GitHub] [calcite] rubenada commented on a change in pull request #1610: [CALCITE-3525] RexSimplify: eliminate redundant rex calls in OR

rubenada commented on a change in pull request #1610: [CALCITE-3525] RexSimplify: eliminate redundant rex calls in OR URL: https://github.com/apache/calcite/pull/1610#discussion_r353731595 ## File path: mongodb/src/test/java/org/apache/calcite/adapter/mongodb/MongoAdapterTest.java

[GitHub] [calcite] ihuzenko commented on a change in pull request #1610: [CALCITE-3525] RexSimplify: eliminate redundant rex calls in OR

ihuzenko commented on a change in pull request #1610: [CALCITE-3525] RexSimplify: eliminate redundant rex calls in OR URL: https://github.com/apache/calcite/pull/1610#discussion_r353726088 ## File path: mongodb/src/test/java/org/apache/calcite/adapter/mongodb/MongoAdapterTest.java

[GitHub] [calcite] yanlin-Lynn opened a new pull request #1630: [CALCITE-3567] Unnest support Map wrapped with RecordType

yanlin-Lynn opened a new pull request #1630: [CALCITE-3567] Unnest support Map wrapped with RecordType URL: https://github.com/apache/calcite/pull/1630 JIRA: [CALCITE-3567](https://issues.apache.org/jira/browse/CALCITE-3567) Currently, UNNEST(RECORDTYPE(ARRAY)) and

[GitHub] [calcite] vlsi closed pull request #1617: [CALCITE-3548] rename ubenchmark/src/main/jmh to ubenchmark/src/jmh/java

vlsi closed pull request #1617: [CALCITE-3548] rename ubenchmark/src/main/jmh to ubenchmark/src/jmh/java URL: https://github.com/apache/calcite/pull/1617 This is an automated message from the Apache Git Service. To respond

[calcite] branch master updated (0e16fe3 -> ecc68ac)

This is an automated email from the ASF dual-hosted git repository. vladimirsitnikov pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/calcite.git. from 0e16fe3 [CALCITE-3536] NPE when executing plan with Coalesce due to wrong NullAs strategy (Jin Xing)

[GitHub] [calcite] rubenada commented on issue #1614: [CALCITE-3546] Improve EnumerableDefaults nested loop join

rubenada commented on issue #1614: [CALCITE-3546] Improve EnumerableDefaults nested loop join URL: https://github.com/apache/calcite/pull/1614#issuecomment-561610527 Thanks @jinxing64 Regarding the tests, I just re-triggered them (2 hours ago). However the "old" Github CI executions

[GitHub] [calcite] jinxing64 commented on issue #1614: [CALCITE-3546] Improve EnumerableDefaults nested loop join

jinxing64 commented on issue #1614: [CALCITE-3546] Improve EnumerableDefaults nested loop join URL: https://github.com/apache/calcite/pull/1614#issuecomment-561604735 +1 This is an automated message from the Apache Git

  1   2   >