Hi Chris, Thanks for the KIP. Overall, it looks good and straightforward to me.
I had a few questions on the new methods 1. I'm not sure if an extension would ever require the task configs. An extension generally should only require the health and current state of the connector which includes the connector config. I was wondering if there is a specific reason it would need task configs. 2. Also, I'm not convinced that kafkaClusterId() belongs to the ConnectClusterState interface. The interface is generally to provide information about the Connect cluster and its information. Also, the kafkaClusterId could potentially change based on whether there is a "producer." or "consumer." prefix, right? Having said that, I would prefer to have connectorConfigs which I think is a great idea and addition to the interface. Let me know what you think. Thanks, Magesh On Sat, Apr 13, 2019 at 9:00 PM Chris Egerton <chr...@confluent.io> wrote: > Hi all, > > I've posted "KIP-454: Expansion of the ConnectClusterState interface", > which proposes that we add provide more information about the Connect > cluster to REST extensions. > > The KIP can be found at > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-454%3A+Expansion+of+the+ConnectClusterState+interface > > I'm eager to hear people's thoughts on this and will appreciate any > feedback. > > Cheers, > > Chris >
