kirktrue commented on code in PR #14670:
URL: https://github.com/apache/kafka/pull/14670#discussion_r1384200253
##########
clients/src/test/java/org/apache/kafka/clients/consumer/internals/LegacyKafkaConsumerTest.java:
##########
@@ -164,7 +163,7 @@
* Note to future authors in this class. If you close the consumer, close with
DURATION.ZERO to reduce the duration of
* the test.
*/
-public class KafkaConsumerTest {
+public class LegacyKafkaConsumerTest {
Review Comment:
I have updated all of the tests to support parameterization. However, about
half of them only support the legacy protocol because when we exercise the new
protocol, the test fails. There is an explanation in the Javadoc of the method
that is used as a 'source' for the parameters that elaborates.
--
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
