ijuma commented on code in PR #12590: URL: https://github.com/apache/kafka/pull/12590#discussion_r1057751426
########## clients/src/test/java/org/apache/kafka/clients/consumer/KafkaConsumerTest.java: ########## @@ -150,6 +150,10 @@ import static org.junit.jupiter.api.Assertions.assertTrue; import static org.junit.jupiter.api.Assertions.fail; +/** + * Note to future authors in this class. If you close the consumer, close with DURATION.ZERO to reduce the duration of the + * test. Review Comment: To clarify, we do recommend avoiding long lines. Ideally, it should be under 105 (this is the middle ground between people who prefer 80 and the ones who prefer 120). -- 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]
