lianetm commented on code in PR #15585:
URL: https://github.com/apache/kafka/pull/15585#discussion_r1555909363
##########
core/src/test/scala/integration/kafka/api/PlaintextConsumerSubscriptionTest.scala:
##########
@@ -39,9 +39,8 @@ class PlaintextConsumerSubscriptionTest extends
AbstractConsumerTest {
* metadata refresh the consumer becomes subscribed to this new topic and
all partitions
* of that topic are assigned to it.
*/
- // TODO: enable this test for the consumer group protocol when support for
pattern subscriptions is implemented.
@ParameterizedTest(name =
TestInfoUtils.TestWithParameterizedQuorumAndGroupProtocolNames)
-
@MethodSource(Array("getTestQuorumAndGroupProtocolParametersClassicGroupProtocolOnly"))
Review Comment:
If we don't use the
`getTestQuorumAndGroupProtocolParametersClassicGroupProtocolOnly` function
anymore we should remove its definition from this file now.
--
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]
