hbtoo opened a new pull request #2358:
URL: https://github.com/apache/calcite/pull/2358
https://issues.apache.org/jira/browse/CALCITE-4514
This is an automated message from the Apache Git Service.
To respond to the message,
danny0405 commented on a change in pull request #2357:
URL: https://github.com/apache/calcite/pull/2357#discussion_r583371130
##
File path: core/src/main/java/org/apache/calcite/rex/RexSimplify.java
##
@@ -2850,11 +2839,16 @@ static RexNode fix(RexBuilder rexBuilder, RexNode
julianhyde commented on pull request #2357:
URL: https://github.com/apache/calcite/pull/2357#issuecomment-786176598
> I have also run the test suite of our downstream project using this PR, no
issue was detected
@rubenada, Thanks for testing this change. It really helps.
julianhyde commented on pull request #2357:
URL: https://github.com/apache/calcite/pull/2357#issuecomment-786175844
@aigor
> I run your pull-request through our test suites (in a downstream project).
> I was not able to detect any serious issues.
Thank you! This helps a
julianhyde commented on a change in pull request #2357:
URL: https://github.com/apache/calcite/pull/2357#discussion_r583132710
##
File path: core/src/main/java/org/apache/calcite/rex/RexSimplify.java
##
@@ -2850,11 +2839,16 @@ static RexNode fix(RexBuilder rexBuilder, RexNode
julianhyde commented on a change in pull request #2357:
URL: https://github.com/apache/calcite/pull/2357#discussion_r583131959
##
File path: core/src/main/java/org/apache/calcite/rex/RexSimplify.java
##
@@ -2840,8 +2829,8 @@ static RexNode fix(RexBuilder rexBuilder, RexNode
rubenada commented on pull request #2357:
URL: https://github.com/apache/calcite/pull/2357#issuecomment-785876439
I have also run the test suite of our downstream project using this PR, no
issue was detected (to be fair we never detected any issue around SEARCH
operator with Calcite 1.26
aigor commented on pull request #2357:
URL: https://github.com/apache/calcite/pull/2357#issuecomment-785866107
Hi @julianhyde , I run your pull-request through our test suites (in a
downstream project).
I was not able to detect any serious issues.
However, with changes from this
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 94502f5 [CALCITE-4506] Upgrade SQLLine to
This is an automated email from the ASF dual-hosted git repository.
rubenql 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 8e7d735d [CALCITE-4437] The Sort rel should
rubenada merged pull request #2302:
URL: https://github.com/apache/calcite/pull/2302
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
11 matches
Mail list logo