Ensure that the executor used for hot deployment of messaging apps is…

… shutdown
loicmathieu · Nov 10, 2022 · cdc35d4 · cdc35d4
1 parent 1d82193
commit cdc35d4
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/...rkus/smallrye/reactivemessaging/runtime/devmode/ReactiveMessagingHotReplacementSetup.java b/...rkus/smallrye/reactivemessaging/runtime/devmode/ReactiveMessagingHotReplacementSetup.java
@@ -1,7 +1,7 @@
 package io.quarkus.smallrye.reactivemessaging.runtime.devmode;
 
 import java.util.concurrent.CompletableFuture;
-import java.util.concurrent.Executor;
+import java.util.concurrent.ExecutorService;
 import java.util.concurrent.Executors;
 import java.util.function.Supplier;
 
@@ -17,14 +17,19 @@ public class ReactiveMessagingHotReplacementSetup implements HotReplacementSetup
 
     private HotReplacementContext context;
     private volatile long nextUpdate;
-    private final Executor executor = Executors.newSingleThreadExecutor();
+    private final ExecutorService executor = Executors.newSingleThreadExecutor();
 
     @Override
     public void setupHotDeployment(HotReplacementContext context) {
         this.context = context;
         DevModeSupportConnectorFactoryInterceptor.register(new OnMessage());
     }
 
+    @Override
+    public void close() {
+        executor.shutdown();
+    }
+
     private class OnMessage implements Supplier<CompletableFuture<Boolean>> {
         @Override
         public CompletableFuture<Boolean> get() {