Merge pull request #2286 from yahoo/gjoranv/catch-exception-in-deconstruct

Catch exception thrown from AbstractComponent.deconstruct.

1 files changed, 11 insertions, 2 deletions

diff --git a/container-disc/src/main/java/com/yahoo/container/jdisc/component/Deconstructor.java b/container-disc/src/main/java/com/yahoo/container/jdisc/component/Deconstructor.java
index 6a2e33db309..ae7599add4c 100644
--- a/container-disc/src/main/java/com/yahoo/container/jdisc/component/Deconstructor.java
+++ b/container-disc/src/main/java/com/yahoo/container/jdisc/component/Deconstructor.java
@@ -12,6 +12,8 @@ import java.util.concurrent.ScheduledExecutorService;
 import java.util.concurrent.TimeUnit;
 import java.util.logging.Logger;
 
+import static java.util.logging.Level.WARNING;
+
 /**
 * @author tonyv
 * @author gv
@@ -54,8 +56,15 @@ public class Deconstructor implements ComponentDeconstructor {
 
         public void run() {
             log.info("Starting deconstruction of " + component);
-            component.deconstruct();
-            log.info("Finished deconstructing " + component);
+            try {
+                component.deconstruct();
+                log.info("Finished deconstructing " + component);
+            } catch (Exception e) {
+                log.log(WARNING, "Exception thrown when deconstructing " + component, e);
+            } catch (Throwable t) {
+                com.yahoo.protect.Process.logAndDie("Error when deconstructing " + component, t);
+            }
         }
+
     }
 }