rreddy-22 commented on code in PR #15150:
URL: https://github.com/apache/kafka/pull/15150#discussion_r1458121585


##########
core/src/test/scala/unit/kafka/admin/ConsumerGroupCommandTest.scala:
##########
@@ -116,6 +123,18 @@ class ConsumerGroupCommandTest extends 
KafkaServerTestHarness {
 }
 
 object ConsumerGroupCommandTest {
+  // We want to test the following combinations:
+  // * ZooKeeper and the classic group protocol.
+  // * KRaft and the classic group protocol.
+  // * KRaft with the new group coordinator enabled and the classic group 
protocol.
+  // * KRaft with the new group coordinator enabled and the consumer group 
protocol.
+  def getTestQuorumAndGroupProtocolParametersAll: 
java.util.stream.Stream[Arguments] = {

Review Comment:
   I wasn't able to re use it so I added the code here but I can double check



-- 
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

Reply via email to