rudy2steiner commented on a change in pull request #7786:
URL: https://github.com/apache/pulsar/pull/7786#discussion_r467660972



##########
File path: pulsar-broker/src/test/resources/log4j2.xml
##########
@@ -19,18 +19,16 @@
 
 -->
 <configuration scan="true">
-<!-- 

Review comment:
       When I try to debug the API, using LoadBalancerTest to launch a multi 
nodes cluster and trying to change the debug level, such as 
org.apache.pulsar.broker.admin. but logback.xml configuration doesn't work and 
found that we actually using log4j2.xml which is configured in parent pom.
   
   I just change the log config file by the way, it's ok if you still think we 
shouldn't change this 




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