FileRequestLogger not using log4j. Intentional?

Hi all,

When we configured query logging to files, we noticed that the FileRequestLogger doesn’t use log4j (https://github.com/druid-io/druid/blob/master/server/src/main/java/io/druid/server/log/FileRequestLogger.java#L71-L74). Is that intentional or something we could get changed with a pull request?

Best regards,

Daniel

I think a better question is:

What is your desired behavior?

Is it something like this? https://github.com/druid-io/druid/pull/3146

Hi Charles,

Thanks a lot for the pull request. That looks like what we were looking for. :slight_smile:

One question we still have is how the logging configuration would look like if we want to keep the query log in a different file than all the other logs. Is that even possible with that “slf4j request logger”?

Best regards,

Daniel

Yes, the default impl for logging in druid is log4j2. You can see an example config at https://github.com/druid-io/druid/blob/druid-0.9.0/common/src/main/resources/log4j2.debug.xml and read more about log4j2 at https://logging.apache.org/log4j/2.x/manual/configuration.html and https://logging.apache.org/log4j/2.x/manual/appenders.html

This should be going out next release