ableegoldman commented on code in PR #12859: URL: https://github.com/apache/kafka/pull/12859#discussion_r1027007714
########## streams/src/test/resources/log4j.properties: ########## @@ -27,7 +27,7 @@ log4j.logger.org.apache.kafka.clients=ERROR # These are the only logs we will likely ever find anything useful in to debug Streams test failures log4j.logger.org.apache.kafka.clients.consumer=INFO log4j.logger.org.apache.kafka.clients.producer=INFO -log4j.logger.org.apache.kafka.streams=INFO +log4j.logger.org.apache.kafka.streams=DEBUG Review Comment: Hm, we might want to be careful about turning on debug logs for all tests -- it's going to explode the log file size and might slow them down, potentially by a lot. Of course debug logs are always useful for debugging (duh) and clearly expand the test coverage to some paths we aren't flexing right now, so it's a tough call Can you actually file a separate ticket for this and extract it to its own PR? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
