flyrain commented on code in PR #12376:
URL: https://github.com/apache/iceberg/pull/12376#discussion_r1972482202
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -4566,7 +4568,9 @@ components:
AuthenticationTimeoutResponse:
description:
- Credentials have timed out. If possible, the client should refresh
credentials and retry.
+ This is an optional status response type that the REST Catalog can
issue when the
+ token has expired. The client MAY request a new access token and retry
the request.
+ 401 UnauthorizedResponse SHOULD be preferred over this response type
on token expiry.
Review Comment:
I like the new descriptions for both status codes. With the new changes, it
seems we actually deprecated it without saying it explicitly. Should we just
deprecate it then? cc @danielcweeks @rdblue
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
