[GitHub] [druid] capistrant commented on a diff in pull request #13020: Provide service specific log4j overrides in containerized deployments

[GitBox <gi...@apache.org>]

capistrant commented on code in PR #13020:
URL: https://github.com/apache/druid/pull/13020#discussion_r966114705


##########
distribution/docker/druid.sh:
##########
@@ -29,6 +29,8 @@
 # Additional env vars:
 # - DRUID_LOG4J -- set the entire log4j.xml verbatim
 # - DRUID_LOG_LEVEL -- override the default log level in default log4j
+# - DRUID_SERVICE_LOG4J -- set the entire service specific log4j.xml verbatim
+# - DRUID_SERVICE_LOG_LEVEL -- override the default log level in the service specific log4j

Review Comment:
   might want to note that this only works if the existing log level in the service log4j file is `info`



-- 
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: commits-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org