SzyWilliam opened a new pull request, #996:
URL: https://github.com/apache/incubator-celeborn/pull/996

   # Fix snapshot loading error
   
   ### What changes were proposed in this pull request?
   Refresh snapshot cache when statemachine takes a new snapshot.
   
   ### Why are the changes needed?
   
   Currently, `getLatestSnapshot` only returns cached snapshot. However, when 
StateMachine takes a new snapshot, it will not update cache, so 
`getLatestSnapshot` cannot get the latest snapshot.
   
   I add the update cache step in the last of `takeSnapshot`. 
   
   
   ### How was this patch tested?
   
   Add new unit tests to verify.
   
   
   /cc @related-reviewer
   
   /assign @main-reviewer
   


-- 
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: [email protected]

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

Reply via email to