[GitHub] [kafka] dajac commented on a change in pull request #11026: KAFKA-13064: refactor ListConsumerGroupOffsetsHandler and tests

2021-07-15 Thread GitBox
dajac commented on a change in pull request #11026: URL: https://github.com/apache/kafka/pull/11026#discussion_r670798256 ## File path: clients/src/main/java/org/apache/kafka/clients/admin/internals/ListConsumerGroupOffsetsHandler.java ## @@ -110,40 +124,96 @@ public String ap

[GitHub] [kafka] dajac commented on a change in pull request #11026: KAFKA-13064: refactor ListConsumerGroupOffsetsHandler and tests

2021-07-15 Thread GitBox
dajac commented on a change in pull request #11026: URL: https://github.com/apache/kafka/pull/11026#discussion_r670737002 ## File path: clients/src/main/java/org/apache/kafka/clients/admin/internals/ListConsumerGroupOffsetsHandler.java ## @@ -82,14 +93,17 @@ public String apiN

[GitHub] [kafka] dajac commented on a change in pull request #11026: KAFKA-13064: refactor ListConsumerGroupOffsetsHandler and tests

2021-07-15 Thread GitBox
dajac commented on a change in pull request #11026: URL: https://github.com/apache/kafka/pull/11026#discussion_r670436565 ## File path: clients/src/main/java/org/apache/kafka/clients/admin/internals/ListConsumerGroupOffsetsHandler.java ## @@ -110,41 +112,97 @@ public String ap

[GitHub] [kafka] dajac commented on a change in pull request #11026: KAFKA-13064: refactor ListConsumerGroupOffsetsHandler and tests

2021-07-13 Thread GitBox
dajac commented on a change in pull request #11026: URL: https://github.com/apache/kafka/pull/11026#discussion_r668864349 ## File path: clients/src/main/java/org/apache/kafka/clients/admin/internals/ListConsumerGroupOffsetsHandler.java ## @@ -110,41 +112,97 @@ public String ap