openstack/deb-python-kafka
Code Issues Proposed changes

A few fixes for offset APIs in 0.8.1

Browse Source
This commit is contained in:
David Arthur
2013-04-02 20:35:22 -04:00
parent fd45ca5933
commit bff6cae885
2 changed files with 1 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
kafka/client.py
View File

@@ -231,7 +231,7 @@ class KafkaClient(object):
def send_offset_fetch_request(self, group, payloads=[], fail_on_error=True, callback=None): def send_offset_fetch_request(self, group, payloads=[], fail_on_error=True, callback=None):
resps = self._send_broker_aware_request(payloads, resps = self._send_broker_aware_request(payloads,
partial(KafkaProtocol.encode_offset_commit_fetch, group=group), partial(KafkaProtocol.encode_offset_fetch_request, group=group),
KafkaProtocol.decode_offset_fetch_response) KafkaProtocol.decode_offset_fetch_response)
out = [] out = []
for resp in resps: for resp in resps:

2
kafka/protocol.py
View File

@@ -354,7 +354,6 @@ class KafkaProtocol(object):
====== ======
data: bytes to decode data: bytes to decode
""" """
data = data[2:] # TODO remove me when versionId is removed
((correlation_id,), cur) = relative_unpack('>i', data, 0) ((correlation_id,), cur) = relative_unpack('>i', data, 0)
(client_id, cur) = read_short_string(data, cur) (client_id, cur) = read_short_string(data, cur)
((num_topics,), cur) = relative_unpack('>i', data, cur) ((num_topics,), cur) = relative_unpack('>i', data, cur)
@@ -398,7 +397,6 @@ class KafkaProtocol(object):
data: bytes to decode data: bytes to decode
""" """
data = data[2:] # TODO remove me when versionId is removed
((correlation_id,), cur) = relative_unpack('>i', data, 0) ((correlation_id,), cur) = relative_unpack('>i', data, 0)
(client_id, cur) = read_short_string(data, cur) (client_id, cur) = read_short_string(data, cur)
((num_topics,), cur) = relative_unpack('>i', data, cur) ((num_topics,), cur) = relative_unpack('>i', data, cur)