rkhachatryan commented on pull request #17203:
URL: https://github.com/apache/flink/pull/17203#issuecomment-918937867


   Thanks for the review @Myasuka 
   > My main concern is how to handle the rescaling case? As same state could 
have different short state id in different subtasks. 
   
   Good point. I addressed this with the backend UUID (discussed in [this 
thread](https://github.com/apache/flink/pull/17203#discussion_r707877114)).
   
   > Since we use key-group as the core design in rescaling, maybe we could 
combine it with group info.
   
   I think using key groups could be more fragile than backend UUID because 
they can be re-assigned multiple times (with multiple rescalings). Besides 
that, that would require more space in snapshot because state name would be 
written the number of times the number of key group assigned.


-- 
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