[GitHub] [calcite] neilnaveen opened a new pull request, #2836: chore: Set permissions for GitHub actions

2022-06-21 Thread GitBox
neilnaveen opened a new pull request, #2836: URL: https://github.com/apache/calcite/pull/2836 Restrict the GitHub token permissions only to the required ones; this way, even if the attackers will succeed in compromising your workflow, they won’t be able to do much. - Included

[GitHub] [calcite] JiajunBernoulli commented on pull request #2615: [CALCITE-4897] Implicit type conversion is not complete for set operation in DML

2022-06-21 Thread GitBox
JiajunBernoulli commented on PR #2615: URL: https://github.com/apache/calcite/pull/2615#issuecomment-1161720140 @danny0405, @rubenada . Thanks for your review. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [calcite-avatica] gabor-hargitai-privitar commented on pull request #148: [CALCITE-2322] Add fetch row count support

2022-06-21 Thread GitBox
gabor-hargitai-privitar commented on PR #148: URL: https://github.com/apache/calcite-avatica/pull/148#issuecomment-1161681249 Hello, what is holding up this feature being merged? It would be very useful. -- This is an automated message from the Apache Git Service. To respond to the

[calcite] branch main updated: [CALCITE-5177] Query loses hint after decorrelation (minor follow-up)

2022-06-21 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 4a01c99e9d [CALCITE-5177] Query loses hint after

[GitHub] [calcite] rubenada merged pull request #2831: [CALCITE-5177] Query loses hint after decorrelation (minor follow-up)

2022-06-21 Thread GitBox
rubenada merged PR #2831: URL: https://github.com/apache/calcite/pull/2831 -- 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] danny0405 commented on a diff in pull request #2831: [CALCITE-5177] Query loses hint after decorrelation (minor follow-up)

2022-06-21 Thread GitBox
danny0405 commented on code in PR #2831: URL: https://github.com/apache/calcite/pull/2831#discussion_r902418910 ## core/src/main/java/org/apache/calcite/plan/RelOptUtil.java: ## @@ -475,6 +475,10 @@ public static RelNode copyRelHints(RelNode originalRel, RelNode newRel) {

[calcite] branch main updated: [CALCITE-4897] Implicit type conversion is not complete for set operation in DML

2022-06-21 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 9a102e77dd [CALCITE-4897] Implicit type

[GitHub] [calcite] rubenada merged pull request #2615: [CALCITE-4897] Implicit type conversion is not complete for set operation in DML

2022-06-21 Thread GitBox
rubenada merged PR #2615: URL: https://github.com/apache/calcite/pull/2615 -- 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] rubenada commented on a diff in pull request #2831: [CALCITE-5177] Query loses hint after decorrelation (minor follow-up)

2022-06-21 Thread GitBox
rubenada commented on code in PR #2831: URL: https://github.com/apache/calcite/pull/2831#discussion_r902239437 ## core/src/main/java/org/apache/calcite/plan/RelOptUtil.java: ## @@ -475,6 +475,10 @@ public static RelNode copyRelHints(RelNode originalRel, RelNode newRel) { *

[calcite] branch main updated: [CALCITE-5157] Query that applies dot operator (field access) to parenthesized expression throws ClassCastException

2022-06-21 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 c8875048e5 [CALCITE-5157] Query that applies dot

[GitHub] [calcite] julianhyde closed pull request #2808: [CALCITE-5157] Fix ClassCastException in checkRollUp with DOT operator

2022-06-21 Thread GitBox
julianhyde closed pull request #2808: [CALCITE-5157] Fix ClassCastException in checkRollUp with DOT operator URL: https://github.com/apache/calcite/pull/2808 -- 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