edoardocomar commented on code in PR #16151: URL: https://github.com/apache/kafka/pull/16151#discussion_r1622769486
########## clients/src/main/java/org/apache/kafka/clients/admin/KafkaAdminClient.java: ########## @@ -4569,7 +4569,7 @@ public ListTransactionsResult listTransactions(ListTransactionsOptions options) public FenceProducersResult fenceProducers(Collection<String> transactionalIds, FenceProducersOptions options) { AdminApiFuture.SimpleAdminApiFuture<CoordinatorKey, ProducerIdAndEpoch> future = FenceProducersHandler.newFuture(transactionalIds); - FenceProducersHandler handler = new FenceProducersHandler(logContext); + FenceProducersHandler handler = new FenceProducersHandler(logContext, requestTimeoutMs); Review Comment: as an override to the AdminClient REQUEST_TIMEOUT_MS_CONFIG ? The options doc says it's on override on API timeout but yes we could use that too -- 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