masteryhx commented on PR #23253: URL: https://github.com/apache/flink/pull/23253#issuecomment-1738497112
@xiangforever2014 Thanks for the detailed clarification. I think remaining too many mapping info in CheckpointCoordinator is not necessary which brings too much unnecessary code path just for sepreated savepoint info. I think the core problem is that current savepoint in detached mode lacks some returned infos to further lookup such as checkpoint id or savepoint id, right ? BTW, This problem also exists in manual checkpoint, so I'd suggest to consider them together. Current REST API could use trigger id to see the status of savepoint which should also work for CLI. So if you pass or generate a trigger id and print it in CLI when detached mode, you could also use it to check the status of savepoint, right ? -- 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]
