Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-11-15 Thread Paolo Patierno
: DevExperience<http://paolopatierno.wordpress.com/> From: isma...@gmail.com on behalf of Ismael Juma Sent: Monday, November 13, 2017 11:44 PM To: dev@kafka.apache.org Cc: Guozhang Wang Subject: Re: [VOTE] KIP-204 : adding records deletion operation t

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-11-13 Thread Ismael Juma
gt; > > > From: Guozhang Wang > Sent: Monday, November 13, 2017 11:11 PM > To: Paolo Patierno > Cc: dev@kafka.apache.org > Subject: Re: [VOTE] KIP-204 : adding records deletion operation to the new > Admin Client API > > Paolo, > >

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-11-13 Thread Paolo Patierno
___ From: Guozhang Wang Sent: Monday, November 13, 2017 11:11 PM To: Paolo Patierno Cc: dev@kafka.apache.org Subject: Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API Paolo, I have indeed voted +1 on the KIP itself, but I thought you only have two binding +

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-11-13 Thread Guozhang Wang
ovember 13, 2017 9:21:07 PM > *To:* Paolo Patierno > *Cc:* dev@kafka.apache.org > > *Subject:* Re: [VOTE] KIP-204 : adding records deletion operation to the > new Admin Client API > > I'm not sure if Ismael's reply on the mailing list is a casted vote for > this KI

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-11-13 Thread Paolo Patierno
PM To: Paolo Patierno Cc: dev@kafka.apache.org Subject: Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API I'm not sure if Ismael's reply on the mailing list is a casted vote for this KIP. @Ismael, could you review the KIP again and cast your vote i

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-11-13 Thread Guozhang Wang
n/paolopatierno> > Blog : DevExperience<http://paolopatierno.wordpress.com/> > > > > From: Colin McCabe > Sent: Friday, November 10, 2017 8:08 PM > To: Guozhang Wang; dev@kafka.apache.org > Subject: Re: [VOTE] KIP-204 : adding records

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-11-13 Thread Paolo Patierno
afka.apache.org Subject: Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API So I have updated the KIP-204<https://cwiki.apache.org/confluence/display/KAFKA/KIP-204+%3A+Adding+records+deletion+operation+to+the+new+Admin+Client+API> using a DeleteReco

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-11-11 Thread Paolo Patierno
om/in/paolopatierno> Blog : DevExperience<http://paolopatierno.wordpress.com/> From: Colin McCabe Sent: Friday, November 10, 2017 8:08 PM To: Guozhang Wang; dev@kafka.apache.org Subject: Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client AP

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-11-10 Thread Colin McCabe
o> > > Linkedin : paolopatierno<http://it.linkedin.com/in/paolopatierno> > > Blog : DevExperience<http://paolopatierno.wordpress.com/> > > > > > > > > From: isma...@gmail.com on behalf of Ismael > > Juma <>

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-11-10 Thread Guozhang Wang
t; Linkedin : paolopatierno<http://it.linkedin.com/in/paolopatierno> > Blog : DevExperience<http://paolopatierno.wordpress.com/> > > > > From: isma...@gmail.com on behalf of Ismael Juma < > ism...@juma.me.uk> > Sent: Friday, November

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-11-10 Thread Paolo Patierno
@ppatierno<http://twitter.com/ppatierno> > Linkedin : paolopatierno<http://it.linkedin.com/in/paolopatierno> > Blog : DevExperience<http://paolopatierno.wordpress.com/> > > > ____ > From: isma...@gmail.com on behalf of Ismael Juma < > ism...@juma.me.uk&

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-11-10 Thread Ismael Juma
xperience<http://paolopatierno.wordpress.com/> > > > > From: isma...@gmail.com on behalf of Ismael Juma < > ism...@juma.me.uk> > Sent: Friday, November 10, 2017 12:33 PM > To: dev@kafka.apache.org > Subject: Re: [VOTE] KIP-204 : ad

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-11-10 Thread Paolo Patierno
lopatierno.wordpress.com/> From: isma...@gmail.com on behalf of Ismael Juma Sent: Friday, November 10, 2017 12:33 PM To: dev@kafka.apache.org Subject: Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API Hi Paolo, The KIP looks good, I ha

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-11-10 Thread Ismael Juma
Hi Paolo, The KIP looks good, I have a couple of comments: 1. partitionsAndOffsets could perhaps be `recordsToDelete`. 2. It seems a bit inconsistent that the argument is `RecordsToDelete`, but the result is just a `Long`. Should the result be `DeleteRecords` or something like that? It could then

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-11-10 Thread Damian Guy
__ > From: Kamal > Sent: Monday, November 6, 2017 1:26 AM > To: dev@kafka.apache.org > Subject: Re: [VOTE] KIP-204 : adding records deletion operation to the new > Admin Client API > > +1 (non-binding) > > On Thu, Nov 2, 2017 at 2:13 PM, Pa

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-11-07 Thread Paolo Patierno
ierno<http://it.linkedin.com/in/paolopatierno> Blog : DevExperience<http://paolopatierno.wordpress.com/> From: Kamal Sent: Monday, November 6, 2017 1:26 AM To: dev@kafka.apache.org Subject: Re: [VOTE] KIP-204 : adding records deletion operation to the

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-11-05 Thread Kamal
rom: Jason Gustafson > Sent: Wednesday, November 1, 2017 10:09 PM > To: dev@kafka.apache.org > Subject: Re: [VOTE] KIP-204 : adding records deletion operation to the new > Admin Client API > > +1 (binding). Just one nit: the KIP doesn't define the DeleteRecordsOptions > object. I se

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-11-02 Thread Paolo Patierno
nkedin.com/in/paolopatierno> Blog : DevExperience<http://paolopatierno.wordpress.com/> From: Jason Gustafson Sent: Wednesday, November 1, 2017 10:09 PM To: dev@kafka.apache.org Subject: Re: [VOTE] KIP-204 : adding records deletion operation to the new

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-11-01 Thread Jason Gustafson
gt; Senior Software Engineer (IoT) @ Red Hat > > > Microsoft MVP on Azure & IoT > > > Microsoft Azure Advisor > > > > > > Twitter : @ppatierno<http://twitter.com/ppatierno> > > > Linkedin : paolopatierno<http://it.linkedin.com/in/paolopatierno

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-11-01 Thread Guozhang Wang
t; > Microsoft Azure Advisor > > > > Twitter : @ppatierno<http://twitter.com/ppatierno> > > Linkedin : paolopatierno<http://it.linkedin.com/in/paolopatierno> > > Blog : DevExperience<http://paolopatierno.wordpress.com/> > > > > > > __

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-10-31 Thread Bill Bejeck
> From: Viktor Somogyi > Sent: Wednesday, October 18, 2017 10:49 AM > To: dev@kafka.apache.org > Subject: Re: [VOTE] KIP-204 : adding records deletion operation to the new > Admin Client API > > +1 (non-binding) > > On Wed, Oct 18, 2017 at 8:23 AM, Manikumar

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-10-31 Thread Paolo Patierno
n.com/in/paolopatierno> Blog : DevExperience<http://paolopatierno.wordpress.com/> From: Viktor Somogyi Sent: Wednesday, October 18, 2017 10:49 AM To: dev@kafka.apache.org Subject: Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-10-18 Thread Viktor Somogyi
+1 (non-binding) On Wed, Oct 18, 2017 at 8:23 AM, Manikumar wrote: > + (non-binding) > > > Thanks, > Manikumar > > On Tue, Oct 17, 2017 at 7:42 AM, Dong Lin wrote: > > > Thanks for the KIP. +1 (non-binding) > > > > On Wed, Oct 11, 2017 at 2:27 AM, Ted Yu wrote: > > > > > +1 > > > > > > On Mon,

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-10-17 Thread Manikumar
+ (non-binding) Thanks, Manikumar On Tue, Oct 17, 2017 at 7:42 AM, Dong Lin wrote: > Thanks for the KIP. +1 (non-binding) > > On Wed, Oct 11, 2017 at 2:27 AM, Ted Yu wrote: > > > +1 > > > > On Mon, Oct 2, 2017 at 10:51 PM, Paolo Patierno > > wrote: > > > > > Hi all, > > > > > > I didn't see

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-10-16 Thread Dong Lin
Thanks for the KIP. +1 (non-binding) On Wed, Oct 11, 2017 at 2:27 AM, Ted Yu wrote: > +1 > > On Mon, Oct 2, 2017 at 10:51 PM, Paolo Patierno > wrote: > > > Hi all, > > > > I didn't see any further discussion around this KIP, so I'd like to start > > the vote for it. > > > > Just for reference :

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-10-11 Thread Ted Yu
+1 On Mon, Oct 2, 2017 at 10:51 PM, Paolo Patierno wrote: > Hi all, > > I didn't see any further discussion around this KIP, so I'd like to start > the vote for it. > > Just for reference : https://cwiki.apache.org/ > confluence/display/KAFKA/KIP-204+%3A+adding+records+ > deletion+operation+to+t

Re: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-10-11 Thread Paolo Patierno
org Subject: [VOTE] KIP-204 : adding records deletion operation to the new Admin Client API Hi all, I didn't see any further discussion around this KIP, so I'd like to start the vote for it. Just for reference : https://cwiki.apache.org/confluence/display/KAFKA/KIP-204+%3A+adding+record

[VOTE] KIP-204 : adding records deletion operation to the new Admin Client API

2017-10-02 Thread Paolo Patierno
Hi all, I didn't see any further discussion around this KIP, so I'd like to start the vote for it. Just for reference : https://cwiki.apache.org/confluence/display/KAFKA/KIP-204+%3A+adding+records+deletion+operation+to+the+new+Admin+Client+API Thanks, Paolo Patierno Senior Software Engineer