+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 > > > > > >
