[calcite] branch main updated: [CALCITE-5274] Improve DocumentBuilderFactory in DiffRepository test class by using secure features

2022-09-08 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 d20fd09a1d [CALCITE-5274] Improve

[GitHub] [calcite] rubenada merged pull request #2898: [CALCITE-5274] Improve DocumentBuilderFactory in DiffRepository test class by using secure features

2022-09-08 Thread GitBox
rubenada merged PR #2898: URL: https://github.com/apache/calcite/pull/2898 -- 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:

[calcite] branch main updated: [CALCITE-5277] Make EnumerableRelImplementor stashedParameters order deterministic to increase BINDABLE_CACHE hit rate

2022-09-08 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 6302e6fa85 [CALCITE-5277] Make

[GitHub] [calcite] rubenada merged pull request #2899: [CALCITE-5277] Make EnumerableRelImplementor stashedParameters order deterministic to increase BINDABLE_CACHE hit rate

2022-09-08 Thread GitBox
rubenada merged PR #2899: URL: https://github.com/apache/calcite/pull/2899 -- 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:

[GitHub] [calcite] Aitozi opened a new pull request, #2901: [CALCITE-5264] Apply hint exclusion check for all rels in the RelOptCall

2022-09-08 Thread GitBox
Aitozi opened a new pull request, #2901: URL: https://github.com/apache/calcite/pull/2901 This PR is meant to apply hint exclusion check for all rels in the RelOptCall. This will help to cover the case that the hint exclusion is attatched not on the rel root. -- This is an automated

[GitHub] [calcite] thomasrebele commented on pull request #2899: [CALCITE-5277] Make EnumerableRelImplementor stashedParameters order deterministic to increase BINDABLE_CACHE hit rate

2022-09-08 Thread GitBox
thomasrebele commented on PR #2899: URL: https://github.com/apache/calcite/pull/2899#issuecomment-1240661122 Seems that we have different definitions of simple code ;) I have the impression that an `IdentityHashMap` and an `ArrayList` is a bit easier to understand than a `LinkedHashMap`

[GitHub] [calcite] dgloeckner opened a new pull request, #2900: CALCITE-5227: add a test case for very wide SELECTs

2022-09-08 Thread GitBox
dgloeckner opened a new pull request, #2900: URL: https://github.com/apache/calcite/pull/2900 This is a test case for https://issues.apache.org/jira/browse/CALCITE-5227 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [calcite] rubenada commented on pull request #2899: [CALCITE-5277] Make EnumerableRelImplementor stashedParameters order deterministic to increase BINDABLE_CACHE hit rate

2022-09-08 Thread GitBox
rubenada commented on PR #2899: URL: https://github.com/apache/calcite/pull/2899#issuecomment-1240569016 @thomasrebele the code is arguably simpler. There is no need to maintain two different data structures (a map plus a list), just a single map fulfills all our needs. IMO this is less

[GitHub] [calcite] thomasrebele commented on pull request #2899: [CALCITE-5277] Make EnumerableRelImplementor stashedParameters order deterministic to increase BINDABLE_CACHE hit rate

2022-09-08 Thread GitBox
thomasrebele commented on PR #2899: URL: https://github.com/apache/calcite/pull/2899#issuecomment-1240487900 Thanks for the PR! What's the advantage of using a `LinkedHashMap` and `Equivalence.Identity` compared to a simple `ArrayList` containing the `ParameterExpression`s in the order

[GitHub] [calcite] rubenada opened a new pull request, #2899: [CALCITE-5277] Make EnumerableRelImplementor stashedParameters order deterministic to increase BINDABLE_CACHE hit rate

2022-09-08 Thread GitBox
rubenada opened a new pull request, #2899: URL: https://github.com/apache/calcite/pull/2899 More info: [CALCITE-5277](https://issues.apache.org/jira/browse/CALCITE-5277) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [calcite] rubenada opened a new pull request, #2898: [CALCITE-5274] Improve DocumentBuilderFactory in DiffRepository test class by using secure features

2022-09-08 Thread GitBox
rubenada opened a new pull request, #2898: URL: https://github.com/apache/calcite/pull/2898 [[CALCITE-5274](https://issues.apache.org/jira/browse/CALCITE-5274)] Improve DocumentBuilderFactory in DiffRepository test class by using secure features -- This is an automated message from the

[calcite] branch main updated: [CALCITE-5251] Support SQL hint for Snapshot

2022-09-08 Thread jhyde
This is an automated email from the ASF dual-hosted git repository. jhyde 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 baeecc82b9 [CALCITE-5251] Support SQL hint for

[GitHub] [calcite] julianhyde merged pull request #2888: [CALCITE-5251] Support SQL hint for Snapshot

2022-09-08 Thread GitBox
julianhyde merged PR #2888: URL: https://github.com/apache/calcite/pull/2888 -- 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: