pnowojski commented on code in PR #26861:
URL: https://github.com/apache/flink/pull/26861#discussion_r2247395574
##########
flink-runtime/src/main/java/org/apache/flink/runtime/taskmanager/Task.java:
##########
@@ -827,6 +829,8 @@ else if (transitionState(current, ExecutionState.FAILED,
t)) {
}
// else fall through the loop and
}
+
+ cleanUpRegistry.close();
Review Comment:
NOTE! Take a look at the try/catch below. There is a change in behaviour.
Now any exception from the cleanup will not be suppressed any more but will be
treated as fatal error (I think correctly).
--
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
