gyfora commented on a change in pull request #31:
URL:
https://github.com/apache/flink-kubernetes-operator/pull/31#discussion_r816979441
##########
File path:
flink-kubernetes-operator/src/main/java/org/apache/flink/kubernetes/operator/reconciler/JobManagerDeploymentStatus.java
##########
@@ -41,14 +38,15 @@
/** JobManager deployment not found, probably not started or killed by
user. */
MISSING;
- public UpdateControl<FlinkDeployment> toUpdateControl(FlinkDeployment
flinkDeployment) {
+ public UpdateControl<FlinkDeployment> toUpdateControl(
+ FlinkDeployment flinkDeployment, int reconcileDelayInSec, int
portCheckIntervalInSec) {
Review comment:
If we have a dedicated configclass for all these values, then it's
enough to pass that and can be changed/extended later
--
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]
