Re: [DISCUSS] KIP-51 - List Connectors REST API

2016-03-22 Thread Ewen Cheslack-Postava
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 wrote: > Hey Ewen, > > Just a quick question. It looks like we're returning a simple array > cont

Re: [DISCUSS] KIP-51 - List Connectors REST API

2016-03-22 Thread Jason Gustafson
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.File

[DISCUSS] KIP-51 - List Connectors REST API

2016-03-22 Thread Ewen Cheslack-Postava
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