ethaden commented on code in PR #14130:
URL: https://github.com/apache/kafka/pull/14130#discussion_r1299353603


##########
clients/src/test/resources/log4j.properties:
##########
@@ -19,3 +19,5 @@ log4j.appender.stdout.layout=org.apache.log4j.PatternLayout
 log4j.appender.stdout.layout.ConversionPattern=[%d] %p %m (%c:%L)%n
 
 log4j.logger.org.apache.kafka=ERROR
+# We are testing for a particular INFO log message in DefaultSslEngineFactory
+log4j.logger.org.apache.kafka.common.security.ssl.DefaultSslEngineFactory=INFO

Review Comment:
   Still couldn't find a way to change the log level programmatically without 
changing any of the (quite reasonable) project configurations such as the 
stylecheck blacklist. But please not that the updated properties file is used 
only during testing. And the log level is changed now only exactly for the 
common name logging wrapper, none of the other classes is affected anymore.



-- 
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: jira-unsubscr...@kafka.apache.org

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

Reply via email to