Michael Blow has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/2962
Change subject: [NO ISSUE] Avoid NPE logging shutdown hooks if shutdown already
in progress
......................................................................
[NO ISSUE] Avoid NPE logging shutdown hooks if shutdown already in progress
Change-Id: Id7102a975bd6e2d653198067bb5b5e7573ded7e7
---
M
hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/ExitUtil.java
1 file changed, 5 insertions(+), 2 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/62/2962/1
diff --git
a/hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/ExitUtil.java
b/hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/ExitUtil.java
index f9d9b1b..e2ae73a 100644
---
a/hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/ExitUtil.java
+++
b/hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/ExitUtil.java
@@ -161,9 +161,12 @@
Field hooksField = clazz.getDeclaredField("hooks");
hooksField.setAccessible(true);
IdentityHashMap hooks = (IdentityHashMap) hooksField.get(null);
- LOGGER.info("the following ({}) shutdown hooks have been
registered: {}", hooks::size, hooks::toString);
+ if (hooks != null) {
+ LOGGER.info("the following ({}) shutdown hooks have been
registered: {}", hooks::size,
+ hooks::toString);
+ }
} catch (Exception e) {
- LOGGER.warn("ignoring exception trying to determine number of
shutdown hooks", e);
+ LOGGER.debug("ignoring exception trying to log shutdown
hooks", e);
}
}
}
--
To view, visit https://asterix-gerrit.ics.uci.edu/2962
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id7102a975bd6e2d653198067bb5b5e7573ded7e7
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: stabilization-f69489
Gerrit-Owner: Michael Blow <[email protected]>
