Re: [PR] [FLINK-34896][table] Migrate CorrelateSortToRankRule to java [flink]

2024-08-28 Thread via GitHub
snuyanzin merged PR #24545: URL: https://github.com/apache/flink/pull/24545 -- 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: issues-unsubscr...@flink.apa

Re: [PR] [FLINK-34896][table] Migrate CorrelateSortToRankRule to java [flink]

2024-08-27 Thread via GitHub
snuyanzin commented on PR #24545: URL: https://github.com/apache/flink/pull/24545#issuecomment-2314345081 @flinkbot run azure -- 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.

Re: [PR] [FLINK-34896][table] Migrate CorrelateSortToRankRule to java [flink]

2024-05-14 Thread via GitHub
snuyanzin commented on code in PR #24545: URL: https://github.com/apache/flink/pull/24545#discussion_r1600671064 ## flink-table/flink-table-planner/src/main/java/org/apache/flink/table/planner/plan/rules/logical/CorrelateSortToRankRule.java: ## @@ -0,0 +1,256 @@ +/* + * Licensed

Re: [PR] [FLINK-34896][table] Migrate CorrelateSortToRankRule to java [flink]

2024-04-08 Thread via GitHub
RyanSkraba commented on code in PR #24545: URL: https://github.com/apache/flink/pull/24545#discussion_r1555815225 ## flink-table/flink-table-planner/src/main/java/org/apache/flink/table/planner/plan/rules/logical/CorrelateSortToRankRule.java: ## @@ -0,0 +1,256 @@ +/* + * License

Re: [PR] [FLINK-34896][table] Migrate CorrelateSortToRankRule to java [flink]

2024-03-20 Thread via GitHub
flinkbot commented on PR #24545: URL: https://github.com/apache/flink/pull/24545#issuecomment-2009564801 ## CI report: * 83bedd5de8ddc74d3095feb937b1728b611db4a1 UNKNOWN Bot commands The @flinkbot bot supports the following commands: - `@flinkbot run azure`

[PR] [FLINK-34896][table] Migrate CorrelateSortToRankRule to java [flink]

2024-03-20 Thread via GitHub
snuyanzin opened a new pull request, #24545: URL: https://github.com/apache/flink/pull/24545 ## What is the purpose of the change A bit reasoning for the change >The reason is that since 1.28.0 ( [CALCITE-4787](https://issues.apache.org/jira/browse/CALCITE-4787) - Move core t