[GitHub] [kafka] jolshan commented on a diff in pull request #13202: KAFKA-14513; Add broker side PartitionAssignor interface

2023-02-09 Thread via GitHub
jolshan commented on code in PR #13202: URL: https://github.com/apache/kafka/pull/13202#discussion_r1101900196 ## group-coordinator/src/main/java/org/apache/kafka/coordinator/group/assignor/PartitionAssignor.java: ## @@ -0,0 +1,43 @@ +/* + * Licensed to the Apache Software Found

[GitHub] [kafka] jolshan commented on a diff in pull request #13202: KAFKA-14513; Add broker side PartitionAssignor interface

2023-02-09 Thread via GitHub
jolshan commented on code in PR #13202: URL: https://github.com/apache/kafka/pull/13202#discussion_r1101899467 ## group-coordinator/src/main/java/org/apache/kafka/coordinator/group/assignor/PartitionAssignor.java: ## @@ -0,0 +1,43 @@ +/* + * Licensed to the Apache Software Found

[GitHub] [kafka] jolshan commented on a diff in pull request #13202: KAFKA-14513; Add broker side PartitionAssignor interface

2023-02-09 Thread via GitHub
jolshan commented on code in PR #13202: URL: https://github.com/apache/kafka/pull/13202#discussion_r1101869241 ## group-coordinator/src/main/java/org/apache/kafka/coordinator/group/assignor/PartitionAssignor.java: ## @@ -0,0 +1,43 @@ +/* + * Licensed to the Apache Software Found

[GitHub] [kafka] jolshan commented on a diff in pull request #13202: KAFKA-14513; Add broker side PartitionAssignor interface

2023-02-09 Thread via GitHub
jolshan commented on code in PR #13202: URL: https://github.com/apache/kafka/pull/13202#discussion_r1101869241 ## group-coordinator/src/main/java/org/apache/kafka/coordinator/group/assignor/PartitionAssignor.java: ## @@ -0,0 +1,43 @@ +/* + * Licensed to the Apache Software Found

[GitHub] [kafka] jolshan commented on a diff in pull request #13202: KAFKA-14513; Add broker side PartitionAssignor interface

2023-02-09 Thread via GitHub
jolshan commented on code in PR #13202: URL: https://github.com/apache/kafka/pull/13202#discussion_r1101863484 ## group-coordinator/src/main/java/org/apache/kafka/coordinator/group/assignor/PartitionAssignor.java: ## @@ -0,0 +1,43 @@ +/* + * Licensed to the Apache Software Found

[GitHub] [kafka] jolshan commented on a diff in pull request #13202: KAFKA-14513; Add broker side PartitionAssignor interface

2023-02-09 Thread via GitHub
jolshan commented on code in PR #13202: URL: https://github.com/apache/kafka/pull/13202#discussion_r1101861603 ## group-coordinator/src/main/java/org/apache/kafka/coordinator/group/assignor/GroupAssignment.java: ## @@ -0,0 +1,57 @@ +/* + * Licensed to the Apache Software Foundat

[GitHub] [kafka] jolshan commented on a diff in pull request #13202: KAFKA-14513; Add broker side PartitionAssignor interface

2023-02-09 Thread via GitHub
jolshan commented on code in PR #13202: URL: https://github.com/apache/kafka/pull/13202#discussion_r1101847493 ## group-coordinator/src/main/java/org/apache/kafka/coordinator/group/assignor/AssignmentSpec.java: ## @@ -0,0 +1,73 @@ +/* + * Licensed to the Apache Software Foundati