Skip to content

Commit

Permalink
Upstream has spoken on retriable errors
Browse files Browse the repository at this point in the history
Add InvalidMessage, and re-order the others to match their definition order.
  • Loading branch information
eapache committed Jan 2, 2016
1 parent bb163bf commit 55f98d5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion async_producer.go
Original file line number Diff line number Diff line change
Expand Up @@ -725,7 +725,7 @@ func (bp *brokerProducer) handleSuccess(sent *produceSet, response *ProduceRespo
}
bp.parent.returnSuccesses(msgs)
// Retriable errors
case ErrUnknownTopicOrPartition, ErrNotLeaderForPartition, ErrLeaderNotAvailable,
case ErrInvalidMessage, ErrUnknownTopicOrPartition, ErrLeaderNotAvailable, ErrNotLeaderForPartition,
ErrRequestTimedOut, ErrNotEnoughReplicas, ErrNotEnoughReplicasAfterAppend:
Logger.Printf("producer/broker/%d state change to [retrying] on %s/%d because %v\n",
bp.broker.ID(), topic, partition, block.Err)
Expand Down

0 comments on commit 55f98d5

Please sign in to comment.