chia7712 commented on code in PR #16963:
URL: https://github.com/apache/kafka/pull/16963#discussion_r1731270580


##########
clients/src/main/java/org/apache/kafka/clients/consumer/internals/AbstractMembershipManager.java:
##########
@@ -269,6 +271,14 @@ public int memberEpoch() {
         return memberEpoch;
     }
 
+    /**
+     * @return Instance ID used by the member when joining the group. If 
non-empty, it will indicate that
+     * this is a static member.
+     */
+    public Optional<String> groupInstanceId() {

Review Comment:
   This method is used by the sub class (`ConsumerHeartbeatRequestManager`) 
only, so maybe we don't need to add it to the parent class 
(`AbstractMembershipManager`)



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

Reply via email to