Apache9 commented on pull request #1708:
URL: https://github.com/apache/hbase/pull/1708#issuecomment-628367457


   For log4j2, there is no way to configuration a category with the grammer like
   
   log4j.rootLogger=INFO,console
   
   Instead, you have to config the log level and log appender separately.
   
   So I changed the start up script a bit to have HBASE_ROOT_LOGGER_LEVEL and 
HBASE_ROOT_LOGGER_APPENDER to specify these two configs separately. This is an 
incompatible change I think.
   
   And I could not find an easy way to purge the dependencies on log4j from 
hadoop, as hadoop is deeply combined with log4j in the metrics system, where 
the org.apache.hadoop.log.metrics.EventCounter is a sub class of the log4j 
AppenderSkeleton.
   
   Just introducing log4j dependency is fine, as we can make sure slf4j-log4j12 
is not in our classpath then no log will be redirect to log4j.But I think these 
metrics will be broken as we will not output logs to these EventCounters.
   
   Maybe we should introduce our own jvm metrics which depends on log4j2?


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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to