[GitHub] [kafka] ableegoldman commented on a change in pull request #9024: MINOR: bump Streams integration test log level to DEBUG

[GitBox <gi...@apache.org>]

ableegoldman commented on a change in pull request #9024:
URL: https://github.com/apache/kafka/pull/9024#discussion_r454721300



##########
File path: streams/src/test/resources/log4j.properties
##########
@@ -18,4 +18,9 @@ log4j.appender.stdout=org.apache.log4j.ConsoleAppender
 log4j.appender.stdout.layout=org.apache.log4j.PatternLayout
 log4j.appender.stdout.layout.ConversionPattern=[%d] %p %m (%c:%L)%n
 
-log4j.logger.org.apache.kafka=INFO
+log4j.logger.kafka=WARN

Review comment:
       We've been discussing this...the way I see it we have three options:
   1) the current proposal (demote brokers to WARN)
   2) just demote ZK to ERROR and see if that helps enough
   3) do a finer grained control of the broker logs, ie demote some less helpful packages to WARN but leave others at INFO
   
   I personally am fine with either 1 or 2 for now, as we can always go back and change it again if it's too little or too much. But I'd rather get fewer broker logs and some Streams logs than no Streams logs at all 🙂 




----------------------------------------------------------------
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:
users@infra.apache.org