Status
Current state: Under discussion.
Discussion thread: here (Not happening yet)
JIRA: here
Motivation
Currently, clients would fail if DNS resolution fails. The application owner will either need to implement retry logic or manually restart the application. This is inconvenient and hard to handle because:
Failed DNS resolution throws a ConfigException, which is not descriptive of the actual problem (the message is fine, but the exception type is misleading); unless the developer tries to parse and match the error message.
It can take minutes before the bootstrap server is registered to the DNS server, and it is reasonable to allow clients to continue to retry.
Public Interfaces
- Users can catch a NetworkException and retry. (Remove the ConfigException)
- Several logging (warn) around failing DNS resolution will be removed.
- DNS lookup will happen on the first poll.
Proposed Changes
Remove the DNS lookup in the client constructor and delegate this task to the NetworkClient poll method, which means the clients won't attempt to resolve for DNS upon starting.
Changes
Client Constructor: Only parse the bootstrap config and validate its format there
- Bootstrap Connection Timeout: A timeout configuration for connecting to the bootstrap server.
NetworkClient:
Bootstrapping should now occur in the poll method before attempting to update the metadata. This includes resolving the addresses and bootstrapping the metadata.
Logs an error message with failed bootstrap process
- If the timeout exceeds, throw a BootstrapConnectionException, which is nonretriable
Configuration Change
bootstrap.connection.timeout.ms
The amount of time clients can try to establish a connection to the bootstrap server and resolve for the IP address. If the time exceeds this value, a BootstrapConnectionException will be thrown.
Note: the default value is up for discussion. It can be 0, which is the same as the current behavior. Exit upon the first retry.
Type: | long |
Default: | 300000 (5 minutes) |
Valid Values: | 0 - LONG_MAX |
Importance: | high |
New Error Type
Name: BootstrapConnectionException extends KafkaException
Message: "Unable to establish a connection to the bootstrap server in {}ms."
Type: Non-retriable.
Compatibility, Deprecation, and Migration Plan
Client Behaviors
Clients won’t attempt to resolve the bootstrap addresses upon initialization.
Clients won’t exit fatally if DNS resolution fails.
KafkaConsumer: Users must poll to retry the lookup if it fails.
KafkaAdminClient: Users will need to resend the request if failing.
KafkaProducer: The sender loop should already be polling continuously.
Exception Handling
Failed DNS resolution will result in NetworkException
Case Study
KafkaConsumer
Case 1: Unable to connect to the bootstrap (For example: misconfiguration)
- The user tries to create a consumer with an invalid bootstrap address. The client was instantiated without an error.
- The user invokes assign() and starts poll()
- The poll returns an empty Consumer Record and logs an error
- The user continues to retry for the configured duration
- The client throws BootstrapConnectionException
Case 2: Transient Network Issue (For example: transient DNS failure)
- The user tries to create a consumer with an invalid bootstrap address. The client was instantiated without an error.
- The user invokes assign() and starts poll()
- The poll returns an empty Consumer Record and logs an error
- The user continues to retry, and the address successfully resolved after x ms
- The poll returns some ConsumerRecrods.
KafkaProducer
Case 1: Unable to connect to the bootstrap (For example: misconfiguration)
- The user instantiates a new producer; the producer creates and starts a sender thread.
- Upon the first runOnce(), client.poll was invoked, which attempts to bootstrap the client. Failure was logged.
- The user tries to produce a message, but because of misconfiguration, the send() will continue to fail on waitOnMetadata until the bootstrap timeout expires
- Once the bootstrap timeout expires, a BootstrapConnectionException is thrown.
Case 2: Transient Network Issue (For example: transient DNS failure)
- The user creates a producer, the sender loop continues to try to connect to the bootstrap server.
- Several send() calls fails with TimeoutException (waitOnMetadata)
- The user retries, eventually successfully send
AdminClient
Case 1: Unable to connect to the bootstrap (For example: misconfiguration)
- The user instantiates a new admin client.
- The AdminClientRunnable thread invokes NetworkClient.poll. The client cannot connect to the bootstrap server; however, the loop continues to poll the NetworkClient.
- Meanwhile, users can make
- Eventually, bootstrap timeout is exhausted. BootstrapConnectionException is thrown.
Case 2: Transient Network Issue (For example: transient DNS failure)
- The user instantiates a new admin client.
- The AdminClientRunnable thread invokes NetworkClient.poll. The client cannot connect to the bootstrap server; however, the loop continues to poll the NetworkClient.
Test Plan
NetworkClient
Test DNS resolution upon its initial poll
Test if the right exception type is thrown
Existing clients (Consumer, Producer, AdminClient)
Test successful bootstrapping upon retrying
Rejected Alternatives
Allow the application owner to specify a retry period. The clients will fail after exceeding the timeout. The default set to 0s, which makes retry an opt-in config.
Pros: Allows users to have more control over how long to retry
Cons: Require a new config; client instantiation can block.
No retry. Let the application owner handle the DNS resolution exception. This means we would still throw a DNSLookupException upon failing.
Pros: No additional config is needed
Cons: This is a behavioral change, and the application owner might need to rewrite the exception handling, i.e. catching the DNS failure logic.
- Not throwing an exception but letting NetworkClient retry on poll.
- Pros: No compatibility break. No additional exception handling logic, the network client will just log the error and continue to retry upon the next poll
- Cons: The discussion thread mentioned that it wouldn't fail upon starting.