diff options
Diffstat (limited to 'container-disc/src')
-rw-r--r-- | container-disc/src/main/java/com/yahoo/container/jdisc/component/Deconstructor.java | 14 |
1 files changed, 8 insertions, 6 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 bb9e10fb6b2..3a153ec3d8a 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 @@ -19,6 +19,8 @@ import java.util.concurrent.ScheduledExecutorService; import java.util.concurrent.TimeUnit; import java.util.logging.Logger; +import static com.yahoo.log.LogLevel.DEBUG; +import static java.util.logging.Level.INFO; import static java.util.logging.Level.SEVERE; import static java.util.logging.Level.WARNING; @@ -50,11 +52,11 @@ public class Deconstructor implements ComponentDeconstructor { } } else if (component instanceof Provider) { // TODO Providers should most likely be deconstructed similarly to AbstractComponent - log.log(LogLevel.DEBUG, () -> "Starting deconstruction of provider " + component); + log.log(DEBUG, () -> "Starting deconstruction of provider " + component); ((Provider<?>) component).deconstruct(); - log.log(LogLevel.DEBUG, () -> "Finished deconstruction of provider " + component); + log.log(DEBUG, () -> "Finished deconstruction of provider " + component); } else if (component instanceof SharedResource) { - log.log(LogLevel.DEBUG, () -> "Releasing container reference to resource " + component); + log.log(DEBUG, () -> "Releasing container reference to resource " + component); // No need to delay release, as jdisc does ref-counting ((SharedResource) component).release(); } @@ -87,10 +89,10 @@ public class Deconstructor implements ComponentDeconstructor { @Override public void run() { for (var component : components) { - log.log(LogLevel.DEBUG, () -> "Starting deconstruction of component " + component); + log.log(DEBUG, () -> "Starting deconstruction of component " + component); try { component.deconstruct(); - log.log(LogLevel.DEBUG, () -> "Finished deconstructing of component " + component); + log.log(DEBUG, () -> "Finished deconstructing of component " + component); } catch (Exception | NoClassDefFoundError e) { // May get class not found due to it being already unloaded log.log(WARNING, "Exception thrown when deconstructing component " + component, e); } catch (Error e) { @@ -110,7 +112,7 @@ public class Deconstructor implements ComponentDeconstructor { // It should now be safe to uninstall the old bundles. for (var bundle : bundles) { try { - log.log(LogLevel.DEBUG, () -> "Uninstalling bundle " + bundle); + log.log(INFO, "Uninstalling bundle " + bundle); bundle.uninstall(); } catch (BundleException e) { log.log(SEVERE, "Could not uninstall bundle " + bundle); |