lianetm commented on code in PR #16963:
URL: https://github.com/apache/kafka/pull/16963#discussion_r1731453376
##########
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:
I agree with the move, and I did think about it the first time but then
thought of KStreams (KIP-1071). I expect Streams groups will need this too, and
we'll probably have to move it back up right? (just sharing what I had in mind,
ok to leave it only where we need it for 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]
