[GitHub] [flink] zoltar9264 commented on pull request #23146: [FLINK-32761][state/backends] Use UUID based on PhysicalStateHandleID…

["zoltar9264 (via GitHub)" <gi...@apache.org>]

zoltar9264 commented on PR #23146:
URL: https://github.com/apache/flink/pull/23146#issuecomment-1667381900

   > Just to double-check: the change does not break the compatibility with the current checkpoints, does it?
   
   Thanks @rkhachatryan for helping to review this PR, I think it is compatible with the current checkpoints : It only changes the key used to register _StreamStateHandle_ with _SharedStateRegistry_, and does not change any thing in checkpoint metadata. When using CLAIM/LEGACY mode to start a job from a retained checkpoint, the retained checkpoint needs to be registered to the _SharedStateRegistry_, but since the registry key will be recalculated at this time, there will be no compatibility issues.


-- 
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-unsubscribe@flink.apache.org

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