rreddy-22 commented on code in PR #13443:
URL: https://github.com/apache/kafka/pull/13443#discussion_r1164463758
##########
group-coordinator/src/main/java/org/apache/kafka/coordinator/group/assignor/GroupAssignment.java:
##########
@@ -35,6 +35,10 @@ public GroupAssignment(
this.members = members;
}
+ public Map<String, MemberAssignment> getMembers() {
Review Comment:
changed in interface changes PR
##########
group-coordinator/src/main/java/org/apache/kafka/coordinator/group/assignor/MemberAssignment.java:
##########
@@ -16,25 +16,28 @@
*/
package org.apache.kafka.coordinator.group.assignor;
-import org.apache.kafka.common.TopicPartition;
+import org.apache.kafka.common.Uuid;
-import java.util.Collection;
+import java.util.Map;
import java.util.Objects;
+import java.util.Set;
+
/**
* The partition assignment for a consumer group member.
*/
public class MemberAssignment {
- /**
- * The target partitions assigned to this member.
- */
- final Collection<TopicPartition> targetPartitions;
-
- public MemberAssignment(
- Collection<TopicPartition> targetPartitions
- ) {
- Objects.requireNonNull(targetPartitions);
- this.targetPartitions = targetPartitions;
+
+ private final Map<Uuid, Set<Integer>> assignmentPerTopic;
+
+
+ public MemberAssignment(Map<Uuid, Set<Integer>> assignmentPerTopic) {
+ Objects.requireNonNull(assignmentPerTopic);
+ this.assignmentPerTopic = assignmentPerTopic;
+ }
+
+ public Map<Uuid, Set<Integer>> getAssignmentPerTopic() {
Review Comment:
changed in interface changes PR
--
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
