Yeah, we can do that. Not sure what we might want to add there, but makes
sense to keep things flexible. Updated the KIP text to reflect this.

-Ewen

On Tue, Mar 22, 2016 at 2:51 PM, Jason Gustafson <ja...@confluent.io> wrote:

> Hey Ewen,
>
> Just a quick question. It looks like we're returning a simple array
> containing the classnames. Would it make sense to return a set of objects
> instead? For example:
>
> [
>   { "class": "org.apache.kafka.connect.file.FileStreamSourceConnector"},
>   { "class": "org.apache.kafka.connect.file.FileStreamSinkConnector" }
> ]
>
> Then we'd be able to include additional fields later without breaking
> compatibility. Other than that, it makes sense to me.
>
> -Jason
>
> On Tue, Mar 22, 2016 at 2:35 PM, Ewen Cheslack-Postava <e...@confluent.io>
> wrote:
>
> > Hi all,
> >
> > It was pointed out that we've been playing a bit fast and loose with API
> > additions in Kafka. I think it's worth discussing a lighter weight
> process
> > for APIs that are still marked unstable, but for the time being we'll add
> > KIPs before adjusting these APIs.
> >
> > To that end, I'd like to discuss (and hopefully quickly move to a vote)
> an
> > API to list connector classes. Here's the KIP:
> >
> >
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-51+-+List+Connectors+REST+API
> >
> > This is a very small addition and already has a patch prepared.
> >
> > -Ewen
> >
>



-- 
Thanks,
Ewen

Reply via email to