pnowojski commented on code in PR #26861:
URL: https://github.com/apache/flink/pull/26861#discussion_r2247946520


##########
flink-runtime/src/main/java/org/apache/flink/runtime/taskmanager/Task.java:
##########
@@ -612,6 +613,7 @@ private void doRun() {
         // need to be undone in the end
         Map<String, Future<Path>> distributedCacheEntries = new HashMap<>();
         TaskInvokable invokable = null;
+        AutoCloseableRegistry cleanUpRegistry = new AutoCloseableRegistry();

Review Comment:
   I was actually hopping for it to be more generic. If someone needs to defer 
some action, this registry could be used.
   
   I could maybe rephrase this comment to:
   
   ```
   // Registry for actions that should be run after the task has already failed 
   ```
   ?



-- 
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

Reply via email to