producer request.timeout.ms needs to be updated in docs

[Jason Rosenberg <jb...@squareup.com>]

Seems the default for this is no 10000
Online doc shows 1500

Just curious, why was this value updated?

Jason

Re: producer request.timeout.ms needs to be updated in docs

[Jun Rao <ju...@gmail.com>]

Thanks for pointing this out. Updated the doc.

The reason for the change is the following. If the timeout is caused by a
problem at the broker, It's actually not very useful to set the timeout too
small. After timing out, the producer is likely to resend the data. This
adds more load to the broker and will make the broker in a worse state. The
only concern with a large timeout is that in case of a machine crash (not
just jvm crash), the client may take up to the timeout time to realize
there is a problem with the broker. However, this is relatively rare. So, a
10 sec timeout is probably still reasonable.

Jun


On Sun, Oct 6, 2013 at 11:57 PM, Jason Rosenberg <jb...@squareup.com> wrote:

> Seems the default for this is no 10000
> Online doc shows 1500
>
> Just curious, why was this value updated?
>
> Jason
>