Re: [PR] [FLINK-35389][State/ForSt] Async list state [flink]

2024-09-03 Thread via GitHub
Zakelly merged PR #25126: URL: https://github.com/apache/flink/pull/25126 -- 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.apach

Re: [PR] [FLINK-35389][State/ForSt] Async list state [flink]

2024-09-02 Thread via GitHub
fredia commented on code in PR #25126: URL: https://github.com/apache/flink/pull/25126#discussion_r1741388216 ## flink-state-backends/flink-statebackend-forst/src/main/java/org/apache/flink/state/forst/ListDelimitedSerializer.java: ## @@ -0,0 +1,94 @@ +/* + * Licensed to the Apa

Re: [PR] [FLINK-35389][State/ForSt] Async list state [flink]

2024-09-02 Thread via GitHub
Zakelly commented on code in PR #25126: URL: https://github.com/apache/flink/pull/25126#discussion_r1740649338 ## flink-state-backends/flink-statebackend-forst/src/main/java/org/apache/flink/state/forst/ListDelimitedSerializer.java: ## @@ -0,0 +1,94 @@ +/* + * Licensed to the Ap

Re: [PR] [FLINK-35389][State/ForSt] Async list state [flink]

2024-09-02 Thread via GitHub
fredia commented on code in PR #25126: URL: https://github.com/apache/flink/pull/25126#discussion_r1740543860 ## flink-state-backends/flink-statebackend-forst/src/main/java/org/apache/flink/state/forst/ListDelimitedSerializer.java: ## @@ -0,0 +1,94 @@ +/* + * Licensed to the Apa

Re: [PR] [FLINK-35389][State/ForSt] Async list state [flink]

2024-08-14 Thread via GitHub
Zakelly commented on PR #25126: URL: https://github.com/apache/flink/pull/25126#issuecomment-2288595953 Conflicts resolved by rebase -- 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 c

Re: [PR] [FLINK-35389][State/ForSt] Async list state [flink]

2024-07-28 Thread via GitHub
Zakelly commented on code in PR #25126: URL: https://github.com/apache/flink/pull/25126#discussion_r169445 ## flink-state-backends/flink-statebackend-forst/src/main/java/org/apache/flink/state/forst/ForStInnerTable.java: ## @@ -73,7 +73,7 @@ public interface ForStInnerTable

Re: [PR] [FLINK-35389][State/ForSt] Async list state [flink]

2024-07-27 Thread via GitHub
jectpro7 commented on code in PR #25126: URL: https://github.com/apache/flink/pull/25126#discussion_r1693958826 ## flink-state-backends/flink-statebackend-forst/src/main/java/org/apache/flink/state/forst/ForStInnerTable.java: ## @@ -73,7 +73,7 @@ public interface ForStInnerTable

Re: [PR] [FLINK-35389][State/ForSt] Async list state [flink]

2024-07-27 Thread via GitHub
flinkbot commented on PR #25126: URL: https://github.com/apache/flink/pull/25126#issuecomment-2254127950 ## CI report: * 6d4c4c0bfe20275155f89d695baf4510236b870e UNKNOWN Bot commands The @flinkbot bot supports the following commands: - `@flinkbot run azure`

[PR] [FLINK-35389][State/ForSt] Async list state [flink]

2024-07-27 Thread via GitHub
Zakelly opened a new pull request, #25126: URL: https://github.com/apache/flink/pull/25126 ## What is the purpose of the change Async List State for ForSt state backend ## Brief change log ## Verifying this change This change added tests under existing test cl