+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 >>>>>> >>>>> >>>> >>>> >> >
signature.asc
Description: OpenPGP digital signature