+1 (binding).

Thanks Chris!

On Wed, Jun 2, 2021 at 11:00 PM Randall Hauch <rha...@gmail.com> wrote:

> Thanks for putting this together, Chris.
>
> Technically, we don't need a new KIP to explicitly remove an API, config,
> etc. that was previously deprecated under an earlier approved KIP. But
> KIP-174 could have been a bit more explicit (note taken for future KIPs)
> that deprecation means "deprecation and future removal", and KIP-738 does
> add a nice migration path for anyone stuck using a non-default internal
> converter.
>
> So I'm +1 (binding) to just wrap this up and remove these configs.
>
> I'd also point out there is a wiki page [1] and dev discussion thread [2]
> highlighting all Connect-related KIPs, including all deprecations. I'll
> update that page to include KIP-736, but it'd be great to have more
> discussion there.
>
> Randall
>
> [1]
> https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=177047362
> [2]
>
> https://lists.apache.org/thread.html/r01e9fa52998a337e435e5d0effca02e74b0552bdec271c1eeca39cd2%40%3Cdev.kafka.apache.org%3E
>
> On Tue, May 18, 2021 at 11:26 AM Ryanne Dolan <ryannedo...@gmail.com>
> wrote:
>
> > +1 (non-binding)
> >
> > Thanks!
> >
> > Ryanne
> >
> > On Tue, May 18, 2021, 6:38 AM Chris Egerton <chr...@confluent.io.invalid
> >
> > wrote:
> >
> > > Hi all,
> > >
> > > I'd like to call for a vote on KIP-738:
> > >
> > >
> >
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-738%3A+Removal+of+Connect%27s+internal+converter+properties
> > >
> > > The discussion thread (which was originally titled with "KIP-736") can
> be
> > > found here:
> > >
> > >
> >
> https://mail-archives.apache.org/mod_mbox/kafka-dev/202105.mbox/%3CCAMdOrUV0bqqs-ry7Q6FkNNn21ZhODTrg2d61zE5WZJw1MpQvSQ%40mail.gmail.com%3E
> > >
> > > Cheers,
> > >
> > > Chris
> > >
> >
>

Reply via email to