openstack/deb-python-kafka
Code Issues Proposed changes

Throw KafkaUnavailableError when no brokers available

Browse Source
This commit is contained in:
Thomas Dimson
2014-01-13 16:39:06 -08:00
parent 9c7b412838
commit a3c3a15460
2 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
kafka/client.py
View File

@@ -8,7 +8,7 @@ from itertools import count
from kafka.common import (ErrorMapping, TopicAndPartition, from kafka.common import (ErrorMapping, TopicAndPartition,
ConnectionError, FailedPayloadsError, ConnectionError, FailedPayloadsError,
BrokerResponseError, PartitionUnavailableError, BrokerResponseError, PartitionUnavailableError,
KafkaRequestError) KafkaUnavailableError, KafkaRequestError)
from kafka.conn import KafkaConnection from kafka.conn import KafkaConnection
from kafka.protocol import KafkaProtocol from kafka.protocol import KafkaProtocol
@@ -78,7 +78,7 @@ class KafkaClient(object):
"trying next server: %s" % (request, conn, e)) "trying next server: %s" % (request, conn, e))
continue continue
raise BrokerResponseError("All servers failed to process request") raise KafkaUnavailableError("All servers failed to process request")
def _send_broker_aware_request(self, payloads, encoder_fn, decoder_fn): def _send_broker_aware_request(self, payloads, encoder_fn, decoder_fn):
""" """

4
kafka/common.py
View File

@@ -78,6 +78,10 @@ class KafkaRequestError(KafkaError):
pass pass
class KafkaUnavailableError(KafkaError):
pass
class BrokerResponseError(KafkaError): class BrokerResponseError(KafkaError):
pass pass