RetryLimit in producer shouldn't be configurable.
remove MaxDeliveryRetries from test file
fix vim barf
Merge pull request #57 from Shopify/retry_limit_shouldnt_be_configurable

RetryLimit in producer shouldn't be configurable.
Return an error in all circumstances where messages are dropped
Include original error value when reporting data loss where possible
Be less lazy about error types: Add a DroppedMessagesError type.
Better definition of DroppedMessagesError
retried as a boolean rather than failures as an int
Merge pull request #59 from Shopify/more_error_reporting

Return an error in all circumstances where messages are dropped
Fixed a logic bug whereby a producer hung after leader election
Force callers to configure MaxBufferedBytes and MaxBufferTime.

Otherwise people will never remember to not set them to zero.
ReplicaNotAvailable is returned from FetchRequests.

It is returned, rather than LeaderNotAvailable, when a replica is
capable of handling your request, but none are available. This applies
(only?) to FetchRequests.
Don't let the caller pass in MaxWaitTime=0.

This mistake is far too easy to make, and rather harmful.
Fix consumer tests
Copy iteration var into block-local to prevent clobbering spawned Gs.
When OutOfBrokers, retry with previously-disconnected brokers.

This is a partial workaround for #15. Really, broker management should
be completely rearchitected, but today is not the day for that.
Yell about un-thought-about config values.
Add a bit more logging to client creation/closing
More verbose logging on failure