Hi Mathieu, I believe you are referring to the connector property "name". This property is a connector config property as opposed to a Connect worker config property. Name is specified per connector instance and not per connector class. The scope of the KIP discussed here (KIP-146) is referring to classloading isolation specifically, and therefore does not seem to be the appropriate forum to discuss the proposed change to the REST API.
I imagine KIP-151 would be more appropriate, but again this KIP is discussing changes to the REST API for Plugins (meaning classes and jars) and not live Connector instances. As I mentioned, "name" is a property of a connector instance not a connector plugin property. I hope this helps. -Konstantine On Fri, May 5, 2017 at 6:44 AM, Mathieu Fenniak < mathieu.fenn...@replicon.com> wrote: > Hi Konstantine, > > As KIP-151 > <https://cwiki.apache.org/confluence/display/KAFKA/KIP- > 151+Expose+Connector+type+in+REST+API> > is currently adding the connector class information to the Kafka Connect > REST API, it might be a nice small detail to augment this KIP to add the > module name to the same API. That would allow a clear audit of where the > code is being loaded. :-) > > Mathieu > > > On Sat, Apr 29, 2017 at 10:16 AM, Konstantine Karantasis < > konstant...@confluent.io> wrote: > > > * Because of KIP number collision, please disregard my previous KIP > > announcement and use this thread for discussion instead * > > > > > > Hi everyone, > > > > we aim to address dependency conflicts in Kafka Connect soon by applying > > class loading isolation. > > > > Feel free to take a look at KIP-146 here: > > > > *https://cwiki.apache.org/confluence/display/KAFKA/KIP- > > 146+-+Classloading+Isolation+in+Connect > > <https://cwiki.apache.org/confluence/display/KAFKA/KIP- > > 146+-+Classloading+Isolation+in+Connect>* > > > > which describes minimal required changes to public interfaces and the > > general implementation approach. > > > > This is a much wanted feature for Kafka Connect. Your feedback is highly > > appreciated. > > > > -Konstantine > > >
