intsmaze commented on issue #7490: restoreFunctionState() add parameter 
getOperatorStateBackend()
URL: https://github.com/apache/flink/pull/7490#issuecomment-454635907
 
 
   > 
   > 
   > -1
   > 
   > I would like to avoid these kind of standalone refactorings, unless the 
change is under some context which makes it necessary.
   > 
   > Moreover, I think the `snapshotFunctionState` has the 
`OperatorStateBackend` parameter because the snapshot context doesn't provide 
access to the state stores. OTOH, the `StateInitializationContext` does provide 
access to the state stores already.
   
   Thank you for your suggestion, I will close the comment later.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to