Apache9 commented on pull request #1708: URL: https://github.com/apache/hbase/pull/1708#issuecomment-799859021
> This looks great. > > What are the implications? Add them as release note. There is the xml format for configuring. What else? Call out your exclude of log4j1. It looks like UI loglevel still works. That's sweet. I was hoping we could back port this to branch-2 as log4j1 has been dead for a long time... We could start a discussion thread on de mailing list to see others opinion? And I think we could also ask the hadoop community to upgrade to log4j2 too, and also, remove the dependencies on log4j or log4j2 when publishing libraries which will be depended by others. ---------------------------------------------------------------- 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]
