dawidwys opened a new pull request #14648:
URL: https://github.com/apache/flink/pull/14648


   ## What is the purpose of the change
   
   It is not yet an entirely ready PR. We should agree how we want to pass the 
information through the `KeyGroupStateHandle` that it is a savepoint. I see two 
options:
   1. Either we add a flag to the `KeyGroupStateHandle`
   2. We add `SavepointKeyGroupStateHandle` that extends from 
`KeyGroupStateHandle`.
   
   My concern with the option 2. is that there would be no real difference 
between the two classes (other than one being a marker for a savepoint). The 
concern for option 1. is that it requires mingling with the java serialization.
   
   ## Verifying this change
   
   


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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to