IntenseFinance/nosqlbench
3
0
Fork 0
mirror of https://github.com/nosqlbench/nosqlbench.git synced 2025-02-25 18:55:28 -06:00
Code Issues Packages Projects Releases Wiki Activity

make logger more robust for early errors

Browse Source
This commit is contained in:
Jonathan Shook 2024-01-12 01:02:39 -06:00
parent 0a2541c593
commit c1f182f306
1 changed files with 10 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
engine-core/src/main/java/io/nosqlbench/engine/core/logging/NBLoggerConfig.java
View File

@ -186,6 +186,13 @@ public class NBLoggerConfig extends ConfigurationFactory {
.add(builder.newAppenderRef("console"))
.addAttribute("additivity", false));
if (sessionName == null) {
sessionName = "LOGINIT";
System.out.println("Session was not set in logger config. If you are getting this error, then it means" +
" that the session has not initialized properly, and that some unexpected error occurred before the logging" +
" system was initialized. Look for errors prior to this.");
}
if (sessionName != null) {
if (!Files.exists(loggerDir)) {
@ -239,10 +246,6 @@ public class NBLoggerConfig extends ConfigurationFactory {
builder.newAppenderRef("SESSION_APPENDER")
.addAttribute("level", fileLevel)
);
} else {
throw new RuntimeException("Session name must be set in logger config. If you are getting this error, then it means" +
" that the session has not initialized properly, and that some unexpected error occurred before the logging" +
" system was initialized. Look for errors prior to this.");
}
rootBuilder.add(
@ -279,7 +282,7 @@ public class NBLoggerConfig extends ConfigurationFactory {
Path logdir = logpath.getParent();
if (Files.exists(logpath)) {
String basename = logpath.getFileName().toString();
String linkname = basename.replace(getSessionName()+"_","");
String linkname = basename.replace(getSessionName() + "_", "");
Path linkPath = logdir.resolve(linkname);
try {
Files.deleteIfExists(linkPath);
@ -448,7 +451,7 @@ public class NBLoggerConfig extends ConfigurationFactory {
private void attachAuxLogger(ConfigurationBuilder<BuiltConfiguration> builder, String loggerName, Level fileLevel) {
String appenderName = loggerName + (("_APPENDER").toUpperCase());
Path logPath = loggerDir.resolve(getFileBase() + "_" + loggerName.toLowerCase() + ".log");
Path linkPath = loggerDir.resolve(loggerName.toLowerCase()+".log");
Path linkPath = loggerDir.resolve(loggerName.toLowerCase() + ".log");
var appender = builder
.newAppender(appenderName, FileAppender.PLUGIN_NAME)
@ -467,7 +470,7 @@ public class NBLoggerConfig extends ConfigurationFactory {
try {
Files.deleteIfExists(linkPath);
Files.createSymbolicLink(linkPath,logPath.getFileName());
Files.createSymbolicLink(linkPath, logPath.getFileName());
} catch (IOException e) {
throw new RuntimeException(e);
}