schongloo commented on PR #969: URL: https://github.com/apache/flink-kubernetes-operator/pull/969#issuecomment-3092468887
> Added some logging related comments. > > We should also handle the `stateless` case a bit more gracefully. A simple stateless example fails with errors like: > > ``` > ... org.apache.flink.kubernetes.operator.exception.UpgradeFailureException: Latest checkpoint not externally addressable, manual recovery required. > ``` > > during upgrades. > > In general if an upgrade would go through successfully for a regular FlinkDeployment it should also go through for a BlueGreenDeployment. While not necessarily related to the `stateless` case. In this situation it's an error I throw when checkpoints are not externally accessible (which is mandatory), the confusing part I throw it during the upgrade only. I should also validate this type of restriction(s) before the initial deployment. I've addressed all the simple refactoring/logging comments, remaining to be done asap: -- 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