ferenc-csaky commented on code in PR #821:
URL:
https://github.com/apache/flink-kubernetes-operator/pull/821#discussion_r1689490037
##########
flink-kubernetes-operator/src/main/java/org/apache/flink/kubernetes/operator/config/FlinkConfigManager.java:
##########
@@ -72,21 +74,26 @@ public class FlinkConfigManager {
private volatile Configuration defaultConfig;
private volatile FlinkOperatorConfiguration defaultOperatorConfiguration;
+ private final boolean snapshotCrdInstalled;
Review Comment:
As we discussed offline, the whole snapshot CRD install check is planned for
1 release to give more meaningful errors for users the first time they upgrade
to an operator version that includes this feature, but will be useless after
that. So I think it would make sense to mark this with a TODO to remove it in
the 1.11 release.
--
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]
