dongnuo123 commented on code in PR #15954:
URL: https://github.com/apache/kafka/pull/15954#discussion_r1603545254
##########
group-coordinator/src/main/java/org/apache/kafka/coordinator/group/GroupMetadataManager.java:
##########
@@ -1244,15 +1244,16 @@ private void throwIfClassicProtocolUnmatched(
String requestProtocolType,
String requestProtocolName
) {
+ String protocolName =
member.supportedClassicProtocols().get().iterator().next().name();
Review Comment:
I'm not sure if it's necessary as we've checked if the member uses the
classic protocol before, but I guess it doesn't hurt to explicitly check the
protocols here. Let me add the validation..
--
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
