Browse Source

Add empty topic error handling

Simon Eskildsen 12 years ago
parent
commit
aa086bfe0b
1 changed files with 3 additions and 0 deletions
  1. 3 0
      client.go

+ 3 - 0
client.go

@@ -184,6 +184,9 @@ func (client *Client) disconnectBroker(broker *Broker) {
 }
 
 func (client *Client) refreshMetadata(topics []string, retries int) error {
+	// Kafka will throw exceptions on an empty topic and not return a proper
+	// error. This handles the case by returning an error instead of sending it
+	// off to Kafka. See: https://github.com/Shopify/sarama/pull/38#issuecomment-26362310
 	for _, topic := range topics {
 		if len(topic) == 0 {
 			return NoSuchTopic