dajac commented on code in PR #13998: URL: https://github.com/apache/kafka/pull/13998#discussion_r1270966763
########## group-coordinator/src/main/java/org/apache/kafka/coordinator/group/assignor/AssignmentTopicMetadata.java: ########## @@ -16,44 +16,83 @@ */ package org.apache.kafka.coordinator.group.assignor; -/** - * Metadata of a topic. - */ -public class AssignmentTopicMetadata { +import org.apache.kafka.common.Uuid; + +import java.util.Collections; +import java.util.Map; +import java.util.Objects; +import java.util.Set; + +public class AssignmentTopicMetadata implements AssignmentTopicDescriber { Review Comment: I see it in the consumer package because this class is about how the TargetAssignmentBuilder uses the PartitionAssignor interface and not really about the PartitionInterface itself. That's right, that assignor package will eventually become public. -- 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