errose28 commented on code in PR #4995:
URL: https://github.com/apache/ozone/pull/4995#discussion_r1259006629


##########
hadoop-ozone/dist/src/shell/ozone/ozone:
##########
@@ -121,6 +121,7 @@ function ozonecmd_case
       ozone_deprecate_envvar HDDS_DN_OPTS OZONE_DATANODE_OPTS
       OZONE_DATANODE_OPTS="${RATIS_OPTS} ${OZONE_DATANODE_OPTS}"
       
OZONE_DATANODE_OPTS="-Dlog4j.configurationFile=${OZONE_CONF_DIR}/dn-audit-log4j2.properties
 ${OZONE_DATANODE_OPTS}"
+      
OZONE_DATANODE_OPTS="-Dlog4j.configurationFile=${OZONE_CONF_DIR}/dn-container-log4j2.properties
 ${OZONE_DATANODE_OPTS}"

Review Comment:
   I have renamed the properties in the container log4j configuration according 
to the 
[docs](https://logging.apache.org/log4j/2.x/manual/configuration.html#CompositeConfiguration)
 so that they will be merged instead of overwritten. After this change I can 
see that both the audit log and container log file are being created inside the 
datanode container.



-- 
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]

Reply via email to