wangyang0918 commented on a change in pull request #11010: [FLINK-15836][k8s]
Throw fatal error in KubernetesResourceManager when the pods watcher is closed
with exception
URL: https://github.com/apache/flink/pull/11010#discussion_r375818015
##########
File path:
flink-kubernetes/src/main/java/org/apache/flink/kubernetes/kubeclient/Fabric8FlinkKubeClient.java
##########
@@ -279,7 +280,7 @@ public void eventReceived(Action action, Pod pod) {
@Override
public void onClose(KubernetesClientException e) {
- LOG.error("The pods watcher is closing.", e);
+ throw new FlinkRuntimeException("The pods
watcher should not be closed.", e);
Review comment:
@tillrohrmann Thanks a lot for your quick review. Yes, you are right. Of
cause, we could not just throw a `FlinkRuntimeException` directly. It needs to
be handled more appropriately. When the pods watcher is closed exceptionally,
it should be handled by the caller. In our case, the`KubernetesResourceManager`
will throw a fatal error.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
