diff options
author | Jeff Widman <jeff@jeffwidman.com> | 2019-03-14 17:34:59 -0700 |
---|---|---|
committer | Dana Powers <dana.powers@gmail.com> | 2019-03-14 17:34:59 -0700 |
commit | 965d21b21387e69c53f583fd76cdcec2a4d0f944 (patch) | |
tree | 81b157cc73458aec84f5fc4687763f6a869c0d97 /kafka | |
parent | 812de351f75beefe73bd9bef55847ab61ccc951d (diff) | |
download | kafka-python-965d21b21387e69c53f583fd76cdcec2a4d0f944.tar.gz |
Error if connections_max_idle_ms not larger than request_timeout_ms (#1688)
Diffstat (limited to 'kafka')
-rw-r--r-- | kafka/consumer/group.py | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/kafka/consumer/group.py b/kafka/consumer/group.py index 531c107..f521891 100644 --- a/kafka/consumer/group.py +++ b/kafka/consumer/group.py @@ -313,11 +313,15 @@ class KafkaConsumer(six.Iterator): new_config, self.config['auto_offset_reset']) self.config['auto_offset_reset'] = new_config + connections_max_idle_ms = self.config['connections_max_idle_ms'] request_timeout_ms = self.config['request_timeout_ms'] fetch_max_wait_ms = self.config['fetch_max_wait_ms'] - if request_timeout_ms <= fetch_max_wait_ms: - raise KafkaConfigurationError("Request timeout (%s) must be larger than fetch-max-wait-ms (%s)" % - (request_timeout_ms, fetch_max_wait_ms)) + if not (fetch_max_wait_ms < request_timeout_ms < connections_max_idle_ms): + raise KafkaConfigurationError( + "connections_max_idle_ms ({}) must be larger than " + "request_timeout_ms ({}) which must be larger than " + "fetch_max_wait_ms ({})." + .format(connections_max_idle_ms, request_timeout_ms, fetch_max_wait_ms)) metrics_tags = {'client-id': self.config['client_id']} metric_config = MetricConfig(samples=self.config['metrics_num_samples'], |