Improve console logging on startup exception

Today we show the exception twice: once by the logger and then again
by the JVM. This is too noisy, and easy to avoid.
This commit is contained in:
Robert Muir 2015-08-18 22:27:39 -04:00
parent 32a097382c
commit f7f7fecafb
1 changed files with 8 additions and 0 deletions

View File

@ -277,11 +277,19 @@ public class Bootstrap {
closeSysError();
}
} catch (Throwable e) {
// disable console logging, so user does not see the exception twice (jvm will show it already)
if (foreground) {
Loggers.disableConsoleLogging();
}
ESLogger logger = Loggers.getLogger(Bootstrap.class);
if (INSTANCE.node != null) {
logger = Loggers.getLogger(Bootstrap.class, INSTANCE.node.settings().get("name"));
}
logger.error("Exception", e);
// re-enable it if appropriate, so they can see any logging during the shutdown process
if (foreground) {
Loggers.enableConsoleLogging();
}
throw e;
}