gyfora commented on code in PR #195: URL: https://github.com/apache/flink-kubernetes-operator/pull/195#discussion_r866940778
########## flink-kubernetes-operator/src/main/java/org/apache/flink/kubernetes/operator/observer/JobStatusObserver.java: ########## @@ -57,7 +55,7 @@ public boolean observe(JobStatus jobStatus, Configuration deployedConfig, CTX ct clusterJobStatuses = new ArrayList<>(flinkService.listJobs(deployedConfig)); } catch (Exception e) { LOG.error("Exception while listing jobs", e); - jobStatus.setState(JOB_STATE_UNKNOWN); + jobStatus.setState(org.apache.flink.api.common.JobStatus.RECONCILING.name()); Review Comment: There are benefits to using one of the Flink JobStatus.states. We don't really care whether the Flink job is actually reconciling (whatever that means) or the operator is reconciling is the flink job that's why I chose it. The name fits :) We can change this later if we feel like it. We could also use simply `null` in these cases. -- 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