Github user tillrohrmann commented on a diff in the pull request:

    https://github.com/apache/flink/pull/5239#discussion_r170026181
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/state/TaskExecutorLocalStateStoresManager.java
 ---
    @@ -89,6 +90,22 @@ public TaskExecutorLocalStateStoresManager(
                                }
                        }
                }
    +
    +           // install a shutdown hook
    +           this.shutdownHook = new 
Thread("TaskExecutorLocalStateStoresManager shutdown hook") {
    --- End diff --
    
    It might make sense to reuse `BlobUtils#addShutdownHook` or to refactor 
this and then reuse.


---

Reply via email to