[GitHub] [calcite] hbtoo opened a new pull request #2358: [CALCITE-4514] Prefer merge new relset into old relset when they are parent set of each other

2021-02-25 Thread GitBox
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,

[GitHub] [calcite] danny0405 commented on a change in pull request #2357: [CALCITE-4446] Implement three-valued logic for SEARCH operator

2021-02-25 Thread GitBox
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

[GitHub] [calcite] julianhyde commented on pull request #2357: [CALCITE-4446] Implement three-valued logic for SEARCH operator

2021-02-25 Thread GitBox
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.

[GitHub] [calcite] julianhyde commented on pull request #2357: [CALCITE-4446] Implement three-valued logic for SEARCH operator

2021-02-25 Thread GitBox
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

[GitHub] [calcite] julianhyde commented on a change in pull request #2357: [CALCITE-4446] Implement three-valued logic for SEARCH operator

2021-02-25 Thread GitBox
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

[GitHub] [calcite] julianhyde commented on a change in pull request #2357: [CALCITE-4446] Implement three-valued logic for SEARCH operator

2021-02-25 Thread GitBox
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

[GitHub] [calcite] rubenada commented on pull request #2357: [CALCITE-4446] Implement three-valued logic for SEARCH operator

2021-02-25 Thread GitBox
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

[GitHub] [calcite] aigor commented on pull request #2357: [CALCITE-4446] Implement three-valued logic for SEARCH operator

2021-02-25 Thread GitBox
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

[calcite] branch master updated: [CALCITE-4506] Upgrade SQLLine to 1.11.0

2021-02-25 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 94502f5 [CALCITE-4506] Upgrade SQLLine to

[calcite] branch master updated: [CALCITE-4437] The Sort rel should be decorrelated even though it has fetch or limit when it is not inside a Correlate (Thomas Rebele)

2021-02-25 Thread rubenql
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

[GitHub] [calcite] rubenada merged pull request #2302: [CALCITE-4437] The Sort rel should be decorrelated even though it has fetch or limit when it is not inside a Correlate (Thomas Rebele)

2021-02-25 Thread GitBox
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