Hey Alex,

Thanks for the KIP. Posting my reply which I accidentally added to another
KIP (before this KIP version was corrected):

I think the timeout controls the maximum allowed time that the consumer
will block for the next record. Maybe the meaning would be clearer with the
more concise name `--timeout`? That also fits with the old consumer which
overrides the `consumer.timeout.ms` property.

By the way, it seems like the default value was intentionally set low for
both the old and new consumers, but I'm not sure of the reason. We could
leave the default as it is if we want to be safe, but increasing it seems
ok to me. Perhaps we could start a little lower, though, say 10 seconds? In
any case, we should make it clear to the user that the timeout was reached.
It's surprising to see only the incomplete reported results following a


On Thu, Apr 5, 2018 at 11:37 AM, Alex Dunayevsky <a.dunayev...@gmail.com>

> Sure, updated the table under 'Public Interfaces' by adding the TimeUnit
> column.
> Thank you
> > In the table under 'Public Interfaces', please add a column with
> TimeUnit.
> > Looks good overall.
> ‌

Reply via email to