Re: [PR] [FLINK-34975][state/forst] Support multiget when db all in local [flink]

2024-09-24 Thread via GitHub
Zakelly commented on code in PR #25363: URL: https://github.com/apache/flink/pull/25363#discussion_r1774522528 ## flink-state-backends/flink-statebackend-forst/src/main/java/org/apache/flink/state/forst/ForStGeneralMultiGetOperation.java: ## @@ -33,39 +37,169 @@ public class Fo

Re: [PR] [FLINK-34975][state/forst] Support multiget when db all in local [flink]

2024-09-24 Thread via GitHub
fredia commented on code in PR #25363: URL: https://github.com/apache/flink/pull/25363#discussion_r1772769493 ## flink-state-backends/flink-statebackend-forst/src/main/java/org/apache/flink/state/forst/ForStGeneralMultiGetOperation.java: ## @@ -40,32 +44,163 @@ public class ForS

Re: [PR] [FLINK-34975][state/forst] Support multiget when db all in local [flink]

2024-09-23 Thread via GitHub
Zakelly commented on code in PR #25363: URL: https://github.com/apache/flink/pull/25363#discussion_r1772552389 ## flink-state-backends/flink-statebackend-forst/src/main/java/org/apache/flink/state/forst/ForStGeneralMultiGetOperation.java: ## @@ -40,32 +44,163 @@ public class For

Re: [PR] [FLINK-34975][state/forst] Support multiget when db all in local [flink]

2024-09-19 Thread via GitHub
flinkbot commented on PR #25363: URL: https://github.com/apache/flink/pull/25363#issuecomment-2362714740 ## CI report: * e003cb105f6d493f5bca7505b2828e043df28766 UNKNOWN Bot commands The @flinkbot bot supports the following commands: - `@flinkbot run azure`

[PR] [FLINK-34975][state/forst] Support multiget when db all in local [flink]

2024-09-19 Thread via GitHub
fredia opened a new pull request, #25363: URL: https://github.com/apache/flink/pull/25363 ## What is the purpose of the change Support multiget when db all in local. ## Brief change log - Add `isRemoteDb` check - Modify `ForStGeneralMultiGetOperation`