dimas-b commented on PR #2197:
URL: https://github.com/apache/polaris/pull/2197#issuecomment-3234687000
> Can we reuse the credential rotate
endpoint(/principals/{principalName}/rotate) instead of having a new endpoint?
My reading of the current Polaris Management API spec is that the "rotate"
endpoint has quite different functionality from "reset". Implementing the
feature proposed in this PR via the old "reset" endpoint could technically be
done (I guess), but I think it will overload the old "reset" functionality too
much and make its logic complicated unnecessarily. I believe adding a new
endpoint is preferable (as implemented in this PR).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@polaris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
