Hi Snehashis,

Thanks for the KIP. I'm +1 (binding) but have one more non-blocking thought
I wanted to share.

On the off chance that an existing plugin is designed to accept a "version"
property, could we either 1) keep passing that property to plugins instead
of stripping it, or 2) rename our new property to something like
"plugin.version"?

Feel free to close the vote (if/when it gets a third binding+1) without
addressing this if you believe the tradeoffs with the existing design are
superior.

Cheers,

Chris

On Mon, Oct 7, 2024, 11:22 Greg Harris <greg.har...@aiven.io.invalid> wrote:

> Hey Snehashis,
>
> Thanks for the KIP! +1 (binding)
>
> Greg
>
> On Fri, Oct 4, 2024 at 10:14 PM Snehashis <snehashisp1...@gmail.com>
> wrote:
>
> > Hi everyone
> >
> > I would like to call a vote for KIP-891. Please take a moment to review
> the
> > proposal and submit your vote. Special thanks to Greg who helped to
> expand
> > this to make it much more broadly useful, and everyone else who
> > participated in both discussion threads.
> >
> > KIP
> > KIP-891: Running multiple versions of Connector plugins - Apache Kafka -
> > Apache Software Foundation
> > <
> >
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-891%3A+Running+multiple+versions+of+Connector+plugins
> > >
> >
> > Thanks
> > Snehashis
> >
>

Reply via email to