masteryhx commented on PR #24173: URL: https://github.com/apache/flink/pull/24173#issuecomment-1909384151
Thanks @JunRuiLee and @WencongLiu pointing out this. @mayuehappy After disscussed offline with @JunRuiLee , I agree that we could not break the change at least for some potential users relying on this (e.g. triggering savepoint with disabled checkpoint). Of course, it's still strange that we still need to start some heavyweight components e.g. CheckpointCoordinator even if users want to disable checkpoint completely. And it's confused that we have different judgement about disabling checkpoint in the code path. So I'd like to hold on this until we found a better solution for this. -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
