tombentley commented on a change in pull request #8311:
URL: https://github.com/apache/kafka/pull/8311#discussion_r420861864
##########
File path:
clients/src/main/java/org/apache/kafka/clients/admin/KafkaAdminClient.java
##########
@@ -1402,6 +1407,15 @@ int numPendingCalls() {
return runnable.pendingCalls.size();
}
+ /**
+ * Fail the given future when a response handler expected a result for an
entity but no result was present.
+ * @param future The future to fail.
+ * @param message The message to fail the future with
+ */
+ private void partialResponse(KafkaFutureImpl<?> future, String message) {
Review comment:
That works for me, especially if this is something you're going to make
more use of for the other response handlers in the admin client.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
