+1 (binding), thanks!

Guozhang


On Thu, May 10, 2018 at 4:41 PM, Ismael Juma <ism...@juma.me.uk> wrote:

> Thanks for the KIP, +1 (binding). A few suggestions:
>
> 1. We normally include the time unit in configs. Not sure if we do it for
> command line parameters though, so can we please verify and make it
> consistent?
> 2. The KIP mentions --polling-loop-timeout and --timeout. Which is it?
> 3. Can we include the description of the new parameter in the KIP? In the
> PR it says "Consumer polling loop timeout", but I think this is a bit
> unclear. What are we actually measuring here?
>
> Ismael
>
> On Mon, Apr 16, 2018 at 2:25 PM Alex Dunayevsky <a.dunayev...@gmail.com>
> wrote:
>
> > Hello friends,
> >
> > Let's start the vote for KIP-281: ConsumerPerformance: Increase Polling
> > Loop Timeout and Make It Reachable by the End User:
> >
> > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> 281%3A+ConsumerPerformance%3A+Increase+Polling+Loop+Timeout+
> and+Make+It+Reachable+by+the+End+User
> >
> > Thank you,
> > Alexander Dunayevsky
> >
>



-- 
-- Guozhang

Reply via email to