Thanks Mickael for the review.
MM1 - Agree, the collision between GET /connectors/expanded and GET
/connectors/{connector} for a connector named "expanded" is an issue.
We'll use GET /connectors-details and a new resource class to avoid
collision with GET /connectors/{connector}.
MM2 - I think that's fair to ignore "expand" in the next major release
and just return the list of connectors. This keeps the API aligned
with the others in terms of not returning a 400 for that parameter. In
the first release we can add a log warning with the existing
functionality.
MM3 - Added the concrete example in the KIP. The response should be
the same as before.
I have added all suggestions to the KIP. Please have another look and
let me know if you have any other feedback.
Thanks
Nilesh
