diff options
author | Dana Powers <dana.powers@rd.io> | 2014-09-21 18:53:26 -0700 |
---|---|---|
committer | Dana Powers <dana.powers@rd.io> | 2014-09-21 18:53:26 -0700 |
commit | eb755b3eadf7448c9716d5add5bdba99ae1490b0 (patch) | |
tree | 38e04cb88bcff99f21007f9d52b33ffc26331d74 /kafka/consumer/simple.py | |
parent | 29f5619385bd26034a21bfb5fd0b827291adb8e4 (diff) | |
download | kafka-python-eb755b3eadf7448c9716d5add5bdba99ae1490b0.tar.gz |
Fix SimpleConsumer timeout behavior in get_messages (Issue 237)
Diffstat (limited to 'kafka/consumer/simple.py')
-rw-r--r-- | kafka/consumer/simple.py | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/kafka/consumer/simple.py b/kafka/consumer/simple.py index dcc71a9..aa4d752 100644 --- a/kafka/consumer/simple.py +++ b/kafka/consumer/simple.py @@ -207,10 +207,11 @@ class SimpleConsumer(Consumer): if not block: # If we're not blocking, break. break - if timeout is not None: - # If we're blocking and have a timeout, reduce it to the - # appropriate value - timeout = max_time - time.time() + + # If we have a timeout, reduce it to the + # appropriate value + if timeout is not None: + timeout = max_time - time.time() # Update and commit offsets if necessary self.offsets.update(new_offsets) |