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

Reply via email to