diff options
author | Dana Powers <dana.powers@gmail.com> | 2019-09-30 10:21:31 -0700 |
---|---|---|
committer | Dana Powers <dana.powers@gmail.com> | 2019-09-30 10:21:31 -0700 |
commit | b5ece5270174e7d767cb79d897c267c1fe2e47f5 (patch) | |
tree | aead2ad0ab08b1c4c2e0a8b9804a3e634ee099e4 | |
parent | 87fb1bb48f82bcaa6c5e1a1edadab2832659801c (diff) | |
download | kafka-python-fix_tests_1_4_7.tar.gz |
Follow up to PR 1782 -- fix testsfix_tests_1_4_7
-rw-r--r-- | kafka/consumer/fetcher.py | 3 | ||||
-rw-r--r-- | test/test_consumer_integration.py | 5 |
2 files changed, 4 insertions, 4 deletions
diff --git a/kafka/consumer/fetcher.py b/kafka/consumer/fetcher.py index f781d4c..5434c36 100644 --- a/kafka/consumer/fetcher.py +++ b/kafka/consumer/fetcher.py @@ -268,6 +268,7 @@ class Fetcher(six.Iterator): start_time = time.time() remaining_ms = timeout_ms + timestamps = copy.copy(timestamps) while remaining_ms > 0: if not timestamps: return {} @@ -294,7 +295,7 @@ class Fetcher(six.Iterator): if refresh_future.succeeded() and isinstance(future.exception, Errors.StaleMetadata): log.debug("Stale metadata was raised, and we now have an updated metadata. Rechecking partition existance") unknown_partition = future.exception.args[0] # TopicPartition from StaleMetadata - if not self._client.cluster.leader_for_partition(unknown_partition): + if self._client.cluster.leader_for_partition(unknown_partition) is None: log.debug("Removed partition %s from offsets retrieval" % (unknown_partition, )) timestamps.pop(unknown_partition) else: diff --git a/test/test_consumer_integration.py b/test/test_consumer_integration.py index c7e2ebf..a2b8f70 100644 --- a/test/test_consumer_integration.py +++ b/test/test_consumer_integration.py @@ -766,12 +766,11 @@ def test_kafka_consumer_offsets_for_time_old(kafka_consumer, topic): @pytest.mark.skipif(env_kafka_version() < (0, 10, 1), reason="Requires KAFKA_VERSION >= 0.10.1") def test_kafka_consumer_offsets_for_times_errors(kafka_consumer_factory, topic): consumer = kafka_consumer_factory(fetch_max_wait_ms=200, - request_timeout_ms=500) + request_timeout_ms=500) tp = TopicPartition(topic, 0) bad_tp = TopicPartition(topic, 100) with pytest.raises(ValueError): consumer.offsets_for_times({tp: -1}) - with pytest.raises(KafkaTimeoutError): - consumer.offsets_for_times({bad_tp: 0}) + assert consumer.offsets_for_times({bad_tp: 0}) == {bad_tp: None} |