vamossagar12 commented on code in PR #14432: URL: https://github.com/apache/kafka/pull/14432#discussion_r1370523927
########## group-coordinator/src/main/java/org/apache/kafka/coordinator/group/GroupMetadataManager.java: ########## @@ -983,27 +1052,125 @@ private CoordinatorResult<ConsumerGroupHeartbeatResponseData, Record> consumerGr return new CoordinatorResult<>(records, response); } + private void replaceStaticMemberInConsumerGroup( + String groupId, + List<Record> records, + ConsumerGroup group, + ConsumerGroupMember member, + ConsumerGroupMember existingStaticMember, + ConsumerGroupMember updatedMember + ) { + // Write tombstones for the departed static member. Follow the same order as the one followed when fencing + // a member + records.add(newCurrentAssignmentTombstoneRecord(group.groupId(), existingStaticMember.memberId())); + records.add(newTargetAssignmentTombstoneRecord(group.groupId(), existingStaticMember.memberId())); + records.add(newMemberSubscriptionTombstoneRecord(group.groupId(), existingStaticMember.memberId())); Review Comment: Done. ########## group-coordinator/src/main/java/org/apache/kafka/coordinator/group/GroupMetadataManager.java: ########## @@ -983,27 +1052,125 @@ private CoordinatorResult<ConsumerGroupHeartbeatResponseData, Record> consumerGr return new CoordinatorResult<>(records, response); } + private void replaceStaticMemberInConsumerGroup( + String groupId, + List<Record> records, + ConsumerGroup group, + ConsumerGroupMember member, + ConsumerGroupMember existingStaticMember, + ConsumerGroupMember updatedMember + ) { + // Write tombstones for the departed static member. Follow the same order as the one followed when fencing + // a member + records.add(newCurrentAssignmentTombstoneRecord(group.groupId(), existingStaticMember.memberId())); + records.add(newTargetAssignmentTombstoneRecord(group.groupId(), existingStaticMember.memberId())); + records.add(newMemberSubscriptionTombstoneRecord(group.groupId(), existingStaticMember.memberId())); + // Cancel all the timers of the departed static member. + cancelConsumerGroupSessionTimeout(group.groupId(), existingStaticMember.memberId()); + cancelConsumerGroupRevocationTimeout(group.groupId(), existingStaticMember.memberId()); Review Comment: Done. -- 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