rkhachatryan commented on a change in pull request #18391:
URL: https://github.com/apache/flink/pull/18391#discussion_r795030756



##########
File path: 
flink-state-backends/flink-statebackend-changelog/src/main/java/org/apache/flink/state/changelog/ChangelogStateBackend.java
##########
@@ -271,7 +273,7 @@ public StateBackend configure(ReadableConfig config, 
ClassLoader classLoader)
                                 keyedStateHandle instanceof 
ChangelogStateBackendHandle
                                         ? (ChangelogStateBackendHandle) 
keyedStateHandle
                                         : new ChangelogStateBackendHandleImpl(
-                                                
singletonList(keyedStateHandle),
+                                                
singletonMap(UUID.randomUUID(), keyedStateHandle),

Review comment:
       > I think you actually mean changelog state-backend does not support 
NO_CLAIM mode yet? 
   
   Yes, that's what I mean (FLINK-25322).
   Currently, it is the default (in master).
   
   And in addition `CLAIM` mode also can not be used because of this issue in 
some scenarios (FLINK-25872). If we aren't going to fix it in this PR then 
please update the docs.




-- 
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.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to