This KIP is now accepted with: - 3 binding +1 - 2 non binding +1 Thanks all
On Fri, 14 Sep 2018 at 21:05 Matthias J. Sax <matth...@confluent.io> wrote: > +1 (binding) > > -Matthias > > On 9/14/18 7:55 AM, Damian Guy wrote: > > +1 (binding) > > > > Thanks > > > > On Fri, 14 Sep 2018 at 10:00 Joan Goyeau <j...@goyeau.com> wrote: > > > >> Ok so we have only one binding vote up to now. I guess we need at least > 3 > >> binding votes. > >> > >> Thanks > >> > >> On Fri, 14 Sep 2018 at 09:59 Joan Goyeau <j...@goyeau.com> wrote: > >> > >>> Matt, This is now all updated. > >>> > >>> On Fri, 7 Sep 2018 at 03:34 Matthias J. Sax <matth...@confluent.io> > >> wrote: > >>> > >>>> Can you please update the KIP accordingly? > >>>> > >>>> It still says "make private" instead of "deprecating" > >>>> > >>>> -Matthias > >>>> > >>>> On 9/6/18 10:07 AM, Attila Sasvári wrote: > >>>>> +1 (non-binding) > >>>>> > >>>>> On Thu, Sep 6, 2018 at 6:38 PM Guozhang Wang <wangg...@gmail.com> > >>>> wrote: > >>>>> > >>>>>> +1 for deprecating and copying the class over to internals. > >>>>>> > >>>>>> On Thu, Sep 6, 2018 at 6:56 AM, Bill Bejeck <bbej...@gmail.com> > >> wrote: > >>>>>> > >>>>>>> +1 > >>>>>>> > >>>>>>> -Bill > >>>>>>> > >>>>>>> On Thu, Sep 6, 2018 at 4:29 AM Joan Goyeau <j...@goyeau.com> > wrote: > >>>>>>> > >>>>>>>> Sournds good, I'll make the deprecation and copy the class over. > >>>>>>>> > >>>>>>>> Thanks > >>>>>>>> > >>>>>>>> On Wed, 5 Sep 2018 at 22:48 John Roesler <j...@confluent.io> > >> wrote: > >>>>>>>> > >>>>>>>>> I'm a +1 (non-binding) because we doubt the class is in use. > >>>>>>>>> > >>>>>>>>> If you decide to copy it to a private version and deprecate the > >>>>>>> original > >>>>>>>>> instead, as Matthias suggested, I would still be a +1. > >>>>>>>>> > >>>>>>>>> Thanks, > >>>>>>>>> -John > >>>>>>>>> > >>>>>>>>> On Sat, Sep 1, 2018 at 6:47 AM Joan Goyeau <j...@goyeau.com> > >> wrote: > >>>>>>>>> > >>>>>>>>>> Hi, > >>>>>>>>>> > >>>>>>>>>> As pointed out in this comment > >>>>>>>>>> https://github.com/apache/kafka/pull/5539#discussion_r212380648 > >>>>>>> "This > >>>>>>>>>> class > >>>>>>>>>> was already defaulted to public visibility, and we can't retract > >> it > >>>>>>>> now, > >>>>>>>>>> without a KIP.", the object FunctionConversions is only of > >> internal > >>>>>>> use > >>>>>>>>> and > >>>>>>>>>> therefore should be private to the lib only so that we can do > >>>>>> changes > >>>>>>>>>> without going through KIP like this one. > >>>>>>>>>> > >>>>>>>>>> KIP: > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>> > >>>>>>>> > https://cwiki.apache.org/confluence/display/KAFKA/KIP-366%3A+Make+ > >>>>>>> FunctionConversions+private > >>>>>>>>>> > >>>>>>>>>> Please make your votes. > >>>>>>>>>> Thanks > >>>>>>>>>> > >>>>>>>>> > >>>>>>>> > >>>>>>> > >>>>>> > >>>>>> > >>>>>> > >>>>>> -- > >>>>>> -- Guozhang > >>>>>> > >>>>> > >>>> > >>>> > >> > > > >