mimaison commented on pull request #10973: URL: https://github.com/apache/kafka/pull/10973#issuecomment-873593049
Yes we should handle errors consistantly but as part of KIP-699, I ensured all existing `KafkaAdminClientTest` tests kept working to avoid changing behaviour, see discussion in https://github.com/apache/kafka/pull/10743#discussion_r653151839. I opened https://issues.apache.org/jira/browse/KAFKA-13012 to address it. This change makes sense but you'll also need to update the `*RetriableErrors()` tests in `KafkaAdminClientTest`. -- 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
