zzbennett commented on code in PR #20061: URL: https://github.com/apache/kafka/pull/20061#discussion_r2178758314
########## group-coordinator/src/main/java/org/apache/kafka/coordinator/group/GroupMetadataManager.java: ########## @@ -485,12 +482,12 @@ GroupMetadataManager build() { /** * The metadata image. */ - private MetadataImage metadataImage; + private CoordinatorMetadataImage metadataImage; /** * The cache for topic hash value by topic name. * A topic hash is calculated when there is a group subscribes to it. - * A topic hash is removed when it's updated in MetadataImage or there is no group subscribes to it. + * A topic hash is removed when it's updated in GroupCoordinatorMetadataImage or there is no group subscribes to it. Review Comment: Found one other place where it was GroupCoordinatorMetadataImage. At first that was what the class was called, but I renamed it to CoordinatorMetadataImage when I realized it was being shared by the Share Group Coordinator too -- 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