Remove slowness warning in console appender

[maxxedev]

Please remove slowness warning in console appender

logback/logback-core/src/main/java/ch/qos/logback/core/ConsoleAppender.java

Lines 91 to 93 in a07cfd5

	addInfo("BEWARE: Writing to the console can be very slow. Avoid logging to the ");
	addInfo("console in production environments, especially in high volume systems.");
	addInfo("See also "+CONSOLE_APPENDER_WARNING_URL);

because most modern applications want to send logs to console:

https://platform9.com/blog/kubernetes-logging-best-practices/
A best practice is to send all application logs to stdout and all error logs to stderr.

https://logz.io/blog/a-practical-guide-to-kubernetes-logging/#simple_example
The best practice is to write your application logs to the standard output (stdout) and standard error (stderr) streams

 
[edit] Additionally, I realized that I saw this info-level status message because there was another warning-level status message in my app.

[Jallibad]

I would also like for this warning to either be removed, to have some way to disable it, or for the documentation to explain what alternative would be preferable in an environment like Kubernetes. As is the warning is frustrating because it results in many additional logs about the logback configuration.

[maxxedev]

One work-around is to add this to logback xml config:

   <statusListener class="ch.qos.logback.core.status.NopStatusListener" />

Or set -Dlogback.statusListenerClass=ch.qos.logback.core.status.NopStatusListener system property.

Or use a ConsoleAppender subclass like this:

import ch.qos.logback.core.ConsoleAppender;

public class ExtendedConsoleAppender<E> extends ConsoleAppender<E> {

    @Override
    public void addInfo(String msg) {
        if ("BEWARE: Writing to the console can be very slow. Avoid logging to the ".equals(msg) ||
            "console in production environments, especially in high volume systems.".equals(msg) ||
            "See also https://logback.qos.ch/codes.html#slowConsole".equals(msg)) {
            return;
        }
        super.addInfo(msg);
    }
}

[ceki]

I should note that the internal status messages are printed at the INFO level and not at the WARN level.

Anyway, how about changing the the wording to the following?

        addInfo("NOTE: Writing to the console can be slow. Try to avoid logging to the ");
        addInfo("console in production environments, especially in high volume systems.");
        addInfo("See also "+CONSOLE_APPENDER_WARNING_URL);

[maxxedev]

Sounds good.

Additionally, I realized that I saw this info-level status message because there was another warning-level status in my app.