+1 from me (binding). On Fri, Aug 24, 2018 at 11:24 AM, Joan Goyeau <[email protected]> wrote:
> Hi, > > As pointed out in this comment #5539 (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. > > Please make your vote. > > On Fri, 24 Aug 2018 at 19:14 John Roesler <[email protected]> wrote: > > > I'm also in favor of this. I don't think it's controversial either. > Should > > we just move to a vote? > > > > On Thu, Aug 23, 2018 at 7:01 PM Guozhang Wang <[email protected]> > wrote: > > > > > +1. > > > > > > On Thu, Aug 23, 2018 at 12:47 PM, Ted Yu <[email protected]> wrote: > > > > > > > +1 > > > > > > > > In the Motivation section, you can quote the comment from pull > request > > so > > > > that reader doesn't have to click through. > > > > > > > > Cheers > > > > > > > > On Thu, Aug 23, 2018 at 12:13 PM Joan Goyeau <[email protected]> > wrote: > > > > > > > > > Hi, > > > > > > > > > > As pointed out in this comment #5539 (comment) > > > > > <https://github.com/apache/kafka/pull/5539#discussion_r212380648> > > 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-363%3A+Make+ > > > > FunctionConversions+private > > > > > > > > > > Thanks > > > > > > > > > > > > > > > > > > > > > -- > > > -- Guozhang > > > > > > -- -- Guozhang
