gyfora commented on code in PR #216:
URL:
https://github.com/apache/flink-kubernetes-operator/pull/216#discussion_r873844978
##########
flink-kubernetes-operator/src/main/java/org/apache/flink/kubernetes/operator/observer/SavepointObserver.java:
##########
@@ -131,9 +136,56 @@ private Optional<String> observeTriggeredSavepointProgress(
LOG.info("Savepoint status updated with latest completed savepoint
info");
currentSavepointInfo.updateLastSavepoint(savepointFetchResult.getSavepoint());
+ updateSavepointHistory(
+ currentSavepointInfo, savepointFetchResult.getSavepoint(),
deployedConfig);
Review Comment:
That makes sense but the savepoint itself could still be recorded in the
history and cleaned up later after upgrade
--
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]
