lbradstreet commented on code in PR #20061:
URL: https://github.com/apache/kafka/pull/20061#discussion_r2188843222


##########
group-coordinator/src/main/java/org/apache/kafka/coordinator/group/GroupCoordinatorService.java:
##########
@@ -2139,7 +2139,7 @@ public void onPartitionsDeleted(
         // At this point the metadata will not have been updated
         // with the deleted topics.
         Set<Uuid> topicIds = topicPartitions.stream()
-            .map(tp -> metadataImage.topics().getTopic(tp.topic()).id())
+            .map(tp -> 
metadataImage.topicId(tp.topic()).orElse(Uuid.ZERO_UUID))

Review Comment:
   One thing that I was thinking about with the ZERO_UUID handling cases is 
that this group coordinator is only supported by clusters that use KRaft, and 
with KRaft you're guaranteed to have a topicId if one exists. So I'm not even 
sure there should be any cases where we want to return ZERO_UUID to clients?



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