[CLOSED] [VOTE] KIP-15 Add a close method with timeout to new producer

2015-03-27 Thread Jiangjie Qin
Thanks everyone for the patient discussion. I really learned a lot from it! KIP-15 is closed with 3 binding +1 and 1 non-binding +1. (Will improve the KIP description as Gwen suggested.) On 3/27/15, 2:05 PM, "Gwen Shapira" wrote: >+1 (non-binding) > >I really like the new API. > >One thing to

Re: [VOTE] KIP-15 Add a close method with timeout to new producer

2015-03-27 Thread Gwen Shapira
+1 (non-binding) I really like the new API. One thing to clarify in the docs: "If sender thread calls close(), it will block forever. So if a close() call is called from sender thread, an error message will be put in the log and close(0) will be called instead." I think we should explicitly ment

Re: [VOTE] KIP-15 Add a close method with timeout to new producer

2015-03-27 Thread Joel Koshy
+1 On Fri, Mar 27, 2015 at 01:34:54PM -0700, Neha Narkhede wrote: > +1 > > Thanks Becket! > > On Fri, Mar 27, 2015 at 1:19 PM, Jay Kreps wrote: > > > +1 > > > > Thanks for your patience Jiangjie! > > > > -Jay > > > > On Fri, Mar 27, 2015 at 11:48 AM, Jiangjie Qin > > wrote: > > > > > > > > >

Re: [VOTE] KIP-15 Add a close method with timeout to new producer

2015-03-27 Thread Neha Narkhede
+1 Thanks Becket! On Fri, Mar 27, 2015 at 1:19 PM, Jay Kreps wrote: > +1 > > Thanks for your patience Jiangjie! > > -Jay > > On Fri, Mar 27, 2015 at 11:48 AM, Jiangjie Qin > wrote: > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-15+-+Add+a+close+method+with+a+timeout+in+the+p

Re: [VOTE] KIP-15 Add a close method with timeout to new producer

2015-03-27 Thread Jay Kreps
+1 Thanks for your patience Jiangjie! -Jay On Fri, Mar 27, 2015 at 11:48 AM, Jiangjie Qin wrote: > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-15+-+Add+a+close+method+with+a+timeout+in+the+producer > > Let us vote again! >

[VOTE] KIP-15 Add a close method with timeout to new producer

2015-03-27 Thread Jiangjie Qin
https://cwiki.apache.org/confluence/display/KAFKA/KIP-15+-+Add+a+close+method+with+a+timeout+in+the+producer Let us vote again!