From 9d18256c66752e57ef9da4d8d119d81b96c37688 Mon Sep 17 00:00:00 2001
From: rongtongjin <794220751@qq.com>
Date: Fri, 27 Sep 2019 15:36:28 +0800
Subject: [PATCH] fix(trace):ignore IllegalStateException when remove shutdown
 hook

If shutdown of the JVM in progress already, we don't need to remove the hook, it will be removed by the JVM.

Close #1491
---
 .../apache/rocketmq/client/trace/AsyncTraceDispatcher.java  | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/client/src/main/java/org/apache/rocketmq/client/trace/AsyncTraceDispatcher.java b/client/src/main/java/org/apache/rocketmq/client/trace/AsyncTraceDispatcher.java
index ca3bcfa2..06a28e44 100644
--- a/client/src/main/java/org/apache/rocketmq/client/trace/AsyncTraceDispatcher.java
+++ b/client/src/main/java/org/apache/rocketmq/client/trace/AsyncTraceDispatcher.java
@@ -216,7 +216,11 @@ public class AsyncTraceDispatcher implements TraceDispatcher {
 
     public void removeShutdownHook() {
         if (shutDownHook != null) {
-            Runtime.getRuntime().removeShutdownHook(shutDownHook);
+            try {
+                Runtime.getRuntime().removeShutdownHook(shutDownHook);
+            } catch (IllegalStateException e) {
+                // ignore - VM is already shutting down
+            }
         }
     }
 
-- 
GitLab