dajac commented on code in PR #18406:
URL: https://github.com/apache/kafka/pull/18406#discussion_r1914532356
##########
group-coordinator/src/main/java/org/apache/kafka/coordinator/group/GroupMetadataManager.java:
##########
@@ -1030,6 +1034,8 @@ private void convertToClassicGroup(
if (joiningMember == null) {
prepareRebalance(classicGroup, String.format("Downgrade group %s
from consumer to classic.", classicGroup.groupId()));
}
+
+ log.info("Converted consumer group {} to classic group.",
consumerGroup.groupId());
Review Comment:
ditto about the format.
##########
group-coordinator/src/main/java/org/apache/kafka/coordinator/group/GroupMetadataManager.java:
##########
@@ -671,7 +671,11 @@ ConsumerGroup getOrMaybeCreateConsumerGroup(
throw new GroupIdNotFoundException(String.format("Consumer group
%s not found.", groupId));
}
- if (group == null || (createIfNotExists &&
maybeDeleteEmptyClassicGroup(group, records))) {
+ if (group == null) {
+ return new ConsumerGroup(snapshotRegistry, groupId, metrics);
+ } else if (createIfNotExists && maybeDeleteEmptyClassicGroup(group,
records)) {
+ log.info("Got an empty classic group {} when getting consumer
group. " +
+ "Removed the old classic group and use an empty consumer group
instead.", groupId);
Review Comment:
Should we use the format of the other log? `[GroupId {}] Converted the empty
classic group to a consumer group.`?
##########
group-coordinator/src/main/java/org/apache/kafka/coordinator/group/GroupMetadataManager.java:
##########
@@ -1110,6 +1116,8 @@ ConsumerGroup convertToConsumerGroup(ClassicGroup
classicGroup, List<Coordinator
scheduleConsumerGroupSessionTimeout(consumerGroup.groupId(),
memberId, member.classicProtocolSessionTimeout().get())
);
+ log.info("Converted classic group {} to consumer group.",
classicGroup.groupId());
Review Comment:
ditto.
##########
group-coordinator/src/main/java/org/apache/kafka/coordinator/group/GroupMetadataManager.java:
##########
@@ -4361,7 +4369,10 @@ CoordinatorResult<Void, CoordinatorRecord>
classicGroupJoinToClassicGroup(
// Group is created if it does not exist and the member id is UNKNOWN.
if member
// is specified but group does not exist, request is rejected with
GROUP_ID_NOT_FOUND
ClassicGroup group;
- maybeDeleteEmptyConsumerGroup(groupId, records);
+ if (maybeDeleteEmptyConsumerGroup(groupId, records)) {
+ log.info("Got an empty consumer group {} when a classic consumer
joins. " +
+ "Removed the old consumer group and use an empty consumer
group instead.", groupId);
Review Comment:
ditto.
--
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]
