I'm going to call this vote since it has enough binding votes and it has been open for a long time. The final tally:
Binding +4: me, Ismael, Rajini, and Guozhang Non-binding +4: Ted, Moshe, Alex, and Dhruvil There were no -1 votes. Thanks to Alexander Dunayevsky for the KIP! Thanks, Jason On Fri, May 11, 2018 at 8:55 AM, Guozhang Wang <wangg...@gmail.com> wrote: > +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 >
