[KAFKA-14782] Implementation Details Different from Documentation (delivery.timeout.ms)

• Type: Bug
• Status: Open
• Priority: Trivial
• Resolution: Unresolved
• Component/s: clients, producer
• Assignee: Unassigned
• Reporter: jangho kwon
• Created: 06/Mar/23 16:12
• Updated: 09/Feb/24 12:23

Description

Hello,

I was checking the value related to delivery.timeout.ms, and I found a relevant document here: https://cwiki.apache.org/confluence/display/KAFKA/KIP-91+Provide+Intuitive+User+Timeouts+in+The+Producer

In the "Validation" section of the document, it is written as follows:

This configuration is backwards compatible. Throw ConfigException for timeouts that don't make sense. (E.g., delivery.timeout.ms < linger.ms + request.timeout.ms + retry.backoff.ms).

However, I noticed that the code does not use the retry.backoff.ms value when checking if delivery.timeout.ms is valid.

If it's not a bug, why was it excluded?

Issue Links

• links to: GitHub Pull Request #13721