[GitHub] [calcite] jonathanswenson opened a new pull request, #2810: [CALCITE-2040] Arrow adapter (continued)

2022-05-18 Thread GitBox
jonathanswenson opened a new pull request, #2810: URL: https://github.com/apache/calcite/pull/2810 Builds on the work done by #2133 and additional commits by @julianhyde on his branch at https://github.com/julianhyde/calcite/tree/2040-arrow Added contributions: + upgrade to arrow

[GitHub] [calcite] chunweilei commented on a diff in pull request #2761: [CALCITE-5073] JoinConditionPushRule cannot infer 'LHS.C1 = LHS.C2' f…

2022-05-18 Thread GitBox
chunweilei commented on code in PR #2761: URL: https://github.com/apache/calcite/pull/2761#discussion_r876568181 ## core/src/main/java/org/apache/calcite/rel/rules/FilterJoinRule.java: ## @@ -225,6 +232,105 @@ protected void perform(RelOptRuleCall call, @Nullable Filter

[calcite] branch dependabot/bundler/site/nokogiri-1.13.5 created (now e10b20662a)

2022-05-18 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/bundler/site/nokogiri-1.13.5 in repository https://gitbox.apache.org/repos/asf/calcite.git at e10b20662a Bump nokogiri from 1.13.4 to 1.13.5 in /site No new revisions were

[GitHub] [calcite] dependabot[bot] opened a new pull request, #2809: Bump nokogiri from 1.13.4 to 1.13.5 in /site

2022-05-18 Thread GitBox
dependabot[bot] opened a new pull request, #2809: URL: https://github.com/apache/calcite/pull/2809 Bumps [nokogiri](https://github.com/sparklemotion/nokogiri) from 1.13.4 to 1.13.5. Release notes Sourced from https://github.com/sparklemotion/nokogiri/releases;>nokogiri's

[GitHub] [calcite] hannerwang commented on pull request #2771: [CALCITE-5089] Allow GROUP BY ALL or DISTINCT set quantifier on GROUPING SETS

2022-05-18 Thread GitBox
hannerwang commented on PR #2771: URL: https://github.com/apache/calcite/pull/2771#issuecomment-1131149074 > BTW, Julian left some comments in the JIRA issue. Could you please address them? Afterward, we can merge it. Thanks @chunweilei , I submit some minor changes, please review.

[GitHub] [calcite] chunweilei commented on a diff in pull request #2803: [CALCITE-5130] AssertionError: "Conversion to relational algebra failed to preserve datatypes" when union VARCHAR literal and C

2022-05-18 Thread GitBox
chunweilei commented on code in PR #2803: URL: https://github.com/apache/calcite/pull/2803#discussion_r876543812 ## core/src/test/java/org/apache/calcite/test/TypeCoercionTest.java: ## @@ -216,7 +216,7 @@ private static ImmutableList combine( sql("select '1' from

[GitHub] [calcite] chunweilei commented on a diff in pull request #2808: [CALCITE-5157] Fix ClassCastException in checkRollUp with DOT operator

2022-05-18 Thread GitBox
chunweilei commented on code in PR #2808: URL: https://github.com/apache/calcite/pull/2808#discussion_r876540880 ## core/src/test/java/org/apache/calcite/test/ReflectiveSchemaTest.java: ## @@ -329,6 +329,16 @@ private int count(ResultSet resultSet) throws SQLException {

[GitHub] [calcite] hannerwang commented on a diff in pull request #2771: [CALCITE-5089] Allow GROUP BY ALL or DISTINCT set quantifier on GROUPING SETS

2022-05-18 Thread GitBox
hannerwang commented on code in PR #2771: URL: https://github.com/apache/calcite/pull/2771#discussion_r876525853 ## testkit/src/main/java/org/apache/calcite/sql/parser/SqlParserTest.java: ## @@ -2122,6 +2122,29 @@ void checkPeriodPredicate(Checker checker) {

[GitHub] [calcite] chunweilei commented on pull request #2800: [CALCITE-3890] Derive IS NOT NULL filter for the inputs of inner join

2022-05-18 Thread GitBox
chunweilei commented on PR #2800: URL: https://github.com/apache/calcite/pull/2800#issuecomment-1130998759 I have squashed commits to a single one. I would like to merge it in the coming days if no objections appear. -- This is an automated message from the Apache Git Service. To respond

[GitHub] [calcite] chunweilei commented on pull request #2800: [CALCITE-3890] Derive IS NOT NULL filter for the inputs of inner join

2022-05-18 Thread GitBox
chunweilei commented on PR #2800: URL: https://github.com/apache/calcite/pull/2800#issuecomment-1130998757 I have squashed commits to a single one. I would like to merge it in the coming days if no objections appear. -- This is an automated message from the Apache Git Service. To respond

[calcite] branch main updated: [CALCITE-5061] Improve recursive application of the field trimming

2022-05-18 Thread rubenql
This is an automated email from the ASF dual-hosted git repository. rubenql pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/calcite.git The following commit(s) were added to refs/heads/main by this push: new dc2e1af5c1 [CALCITE-5061] Improve recursive

[GitHub] [calcite] rubenada merged pull request #2806: [CALCITE-5061] Improve recursive application of the field trimming

2022-05-18 Thread GitBox
rubenada merged PR #2806: URL: https://github.com/apache/calcite/pull/2806 -- 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 to the specific comment. To unsubscribe, e-mail: