dongnuo123 commented on code in PR #16026: URL: https://github.com/apache/kafka/pull/16026#discussion_r1613860946
########## group-coordinator/src/main/java/org/apache/kafka/coordinator/group/classic/ClassicGroup.java: ########## @@ -869,11 +860,7 @@ public void validateOffsetCommit( } /** - * Validates the OffsetFetch request. - * - * @param memberId The member id. This is not provided for classic groups. - * @param memberEpoch The member epoch for consumer groups. This is not provided for classic groups. Review Comment: Not sure if we want to keep these? `This is not provided for classic groups` is not mentioned in the comment in the Group interface -- 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