summaryrefslogtreecommitdiff
path: root/kafka/client.py
diff options
context:
space:
mode:
authorDana Powers <dana.powers@rd.io>2015-06-09 21:57:26 -0700
committerDana Powers <dana.powers@rd.io>2015-06-09 21:57:26 -0700
commitd6585483e0851e85a639b954b59807606792f516 (patch)
tree65bc59a5130c7f16dea2b74ac28e5ab38d17a540 /kafka/client.py
parentbedaafa8046ed172b64f8ea4d444fcd8dbb8ae92 (diff)
downloadkafka-python-d6585483e0851e85a639b954b59807606792f516.tar.gz
KafkaClient: log responses by requestId for debugging
Diffstat (limited to 'kafka/client.py')
-rw-r--r--kafka/client.py5
1 files changed, 4 insertions, 1 deletions
diff --git a/kafka/client.py b/kafka/client.py
index 6b07acd..5c869b9 100644
--- a/kafka/client.py
+++ b/kafka/client.py
@@ -189,10 +189,12 @@ class KafkaClient(object):
# No exception, try to get response
else:
- # decoder_fn=None signal that the server is expected to not
+ # decoder_fn=None signal that the server is expected to not
# send a response. This probably only applies to
# ProduceRequest w/ acks = 0
if decoder_fn is None:
+ log.debug('Request %s does not expect a response '
+ '(skipping conn.recv)', requestId)
for payload in payloads:
responses_by_broker[broker].append(None)
continue
@@ -211,6 +213,7 @@ class KafkaClient(object):
else:
for payload_response in decoder_fn(response):
responses_by_broker[broker].append(payload_response)
+ log.debug('Response %s: %s', requestId, responses_by_broker[broker])
# Connection errors generally mean stale metadata
# although sometimes it means incorrect api request