TSFenwick commented on code in PR #14094:
URL: https://github.com/apache/druid/pull/14094#discussion_r1169312057
##########
indexing-service/src/main/java/org/apache/druid/indexing/common/tasklogs/ConsoleLoggingEnforcementConfigurationFactory.java:
##########
@@ -117,16 +123,20 @@ private Appender findConsoleAppender()
}
/**
- * remove all appenders from a logger and append a console appender to it
+ * Ensure there is a console logger defined. Without a console logger peon
logs wont be able to be stored in deep storage
*/
private void applyConsoleAppender(LoggerConfig logger, Appender
consoleAppender)
{
- if (logger.getAppenderRefs().size() == 1
- &&
logger.getAppenderRefs().get(0).getRef().equals(consoleAppender.getName())) {
- // this logger has only one appender and its the console appender
- return;
+ // Alter log level for this class to be warning. This needs to happen
because the logger is using the default
+ // config, which is level error and appends to console, since the logger
is being configured here.
+
Configurator.setLevel(LogManager.getLogger(ConsoleLoggingEnforcementConfigurationFactory.class).getName(),
Level.WARN);
Review Comment:
done. needed to alter the druid logger class to make getName available.
testing out the change to make sure it still works as expected
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
