Looks good overall. Currently lookupKafkaClusterId() is called synchronously. Have you considered making the call asynchronous (normally the GET / request comes sometime after worker start) ?
Thanks On Mon, Dec 11, 2017 at 3:40 PM, Ewen Cheslack-Postava <e...@confluent.io> wrote: > I'd like to start discussion on a simple KIP to expose Kafka cluster ID > info in the Connect REST API: > > https://cwiki.apache.org/confluence/display/KAFKA/KIP- > 238%3A+Expose+Kafka+cluster+ID+in+Connect+REST+API > > Hopefully straightforward, though there are some details on how this > affects startup behavior that might warrant discussion. > > -Ewen >
