kirktrue commented on PR #18702: URL: https://github.com/apache/kafka/pull/18702#issuecomment-2623352548
`request.timeout.ms` is designed to apply to _each_ discrete network request whereas the close timeout applies to the _entire_ `KafkaConsumer.close()` call. If the user provides a timeout of 60 seconds to `close()`, why should it give up after only 30 seconds? As a parallel, consider the relationship between `request.timeout.ms` and [`delivery.timeout.ms`](https://kafka.apache.org/documentation/#producerconfigs_delivery.timeout.ms) in the `KafkaProducer.send()` call. In that case, it's `delivery.timeout.ms`, not `request.timeout.ms`, that serves as the deadline by which all of the network calls in `send()` must complete. That said, let's fix this gap with the solution as proposed. No one has complained about the behavior of the existing consumer timeout. I have no intention of dying on this hill; I just personally find it confusing 😄 -- 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
