gyfora commented on code in PR #633:
URL:
https://github.com/apache/flink-kubernetes-operator/pull/633#discussion_r1266829192
##########
flink-kubernetes-operator-api/src/main/java/org/apache/flink/kubernetes/operator/api/status/CommonStatus.java:
##########
@@ -93,4 +95,10 @@ public ResourceLifecycleState getLifecycleState() {
return ResourceLifecycleState.DEPLOYED;
}
+
+ /**
+ * Internal flag to signal that due to some condition we need to schedule
a new reconciliation
+ * loop immediately. For example autoscaler overrides have changed and we
need to apply them.
+ */
+ @JsonIgnore @Internal private boolean immediateReconciliationNeeded =
false;
Review Comment:
This is the same behaviour we have for all upgrades. Rescheduling through
the operator allows us to avoid potential deadlocks, have a fair reconciliation
queue + to observe one more time before we execute a new action. This allows us
to pick up any spec change that may have come in in the meantime.
--
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]
