[GitHub] [kafka] C0urante commented on a diff in pull request #14314: KAFKA-15425: Fail fast in Admin::listOffsets when topic (but not partition) metadata is not found

2023-09-07 Thread via GitHub
C0urante commented on code in PR #14314: URL: https://github.com/apache/kafka/pull/14314#discussion_r1318868378 ## clients/src/test/java/org/apache/kafka/clients/admin/KafkaAdminClientTest.java: ## @@ -5633,20 +5643,39 @@ public void testListOffsetsMetadataNonRetriableErrors()

[GitHub] [kafka] C0urante commented on a diff in pull request #14314: KAFKA-15425: Fail fast in Admin::listOffsets when topic (but not partition) metadata is not found

2023-09-07 Thread via GitHub
C0urante commented on code in PR #14314: URL: https://github.com/apache/kafka/pull/14314#discussion_r1318748685 ## clients/src/test/java/org/apache/kafka/clients/admin/KafkaAdminClientTest.java: ## @@ -5634,16 +5637,31 @@ public void testListOffsetsMetadataNonRetriableErrors()

[GitHub] [kafka] C0urante commented on a diff in pull request #14314: KAFKA-15425: Fail fast in Admin::listOffsets when topic (but not partition) metadata is not found

2023-09-07 Thread via GitHub
C0urante commented on code in PR #14314: URL: https://github.com/apache/kafka/pull/14314#discussion_r1318720780 ## clients/src/main/java/org/apache/kafka/clients/admin/internals/PartitionLeaderStrategy.java: ## @@ -72,6 +79,12 @@ private void handleTopicError( ) {