AndrewJSchofield commented on code in PR #14758:
URL: https://github.com/apache/kafka/pull/14758#discussion_r1403093845
##########
core/src/test/scala/kafka/utils/TestInfoUtils.scala:
##########
@@ -39,6 +39,12 @@ object TestInfoUtils {
} else {
throw new RuntimeException(s"Unknown quorum value")
}
+ } else if (testInfo.getDisplayName().contains("groupProtocol=")) {
Review Comment:
It's because the parameterized tests in `PlaintextConsumerTest` choose
between "generic" and "consumer". The new consumer ("consumer") is only
supported for KRaft clusters, so it's necessary to make the test use KRaft in
this instance.
--
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
