You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flink.apache.org by ch...@apache.org on 2022/11/24 08:18:42 UTC

[flink] branch release-1.16 updated: [FLINK-26037][coordination] TMRunner waits for metric registry shutdown

This is an automated email from the ASF dual-hosted git repository.

chesnay pushed a commit to branch release-1.16
in repository https://gitbox.apache.org/repos/asf/flink.git


The following commit(s) were added to refs/heads/release-1.16 by this push:
     new 39faab7202d [FLINK-26037][coordination] TMRunner waits for metric registry shutdown
39faab7202d is described below

commit 39faab7202db16067a8e9b13c2cd8dc420d900b8
Author: Chesnay Schepler <ch...@apache.org>
AuthorDate: Fri Nov 18 12:51:27 2022 +0100

    [FLINK-26037][coordination] TMRunner waits for metric registry shutdown
---
 .../java/org/apache/flink/runtime/taskexecutor/TaskManagerRunner.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/flink-runtime/src/main/java/org/apache/flink/runtime/taskexecutor/TaskManagerRunner.java b/flink-runtime/src/main/java/org/apache/flink/runtime/taskexecutor/TaskManagerRunner.java
index 0c2b82ff2b5..762695c67ef 100644
--- a/flink-runtime/src/main/java/org/apache/flink/runtime/taskexecutor/TaskManagerRunner.java
+++ b/flink-runtime/src/main/java/org/apache/flink/runtime/taskexecutor/TaskManagerRunner.java
@@ -380,7 +380,7 @@ public class TaskManagerRunner implements FatalErrorHandler {
 
             if (metricRegistry != null) {
                 try {
-                    metricRegistry.shutdown();
+                    terminationFutures.add(metricRegistry.shutdown());
                 } catch (Exception e) {
                     exception = ExceptionUtils.firstOrSuppressed(e, exception);
                 }