danielcweeks commented on code in PR #8976: URL: https://github.com/apache/iceberg/pull/8976#discussion_r1473325248
########## core/src/test/java/org/apache/iceberg/rest/RESTCatalogAdapter.java: ########## @@ -105,6 +105,8 @@ enum HTTPMethod { enum Route { TOKENS(HTTPMethod.POST, "v1/oauth/tokens", null, OAuthTokenResponse.class), + SEPARATE_AUTH_TOKENS_URI( Review Comment: This looks like it was added to facilitate with the tests, but the CatalogAdapter servers a larger purpose than just the tests. I'm not confident we want to handle it like this. -- 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...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org