nastra commented on code in PR #9940:
URL: https://github.com/apache/iceberg/pull/9940#discussion_r1665314780
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -191,7 +220,8 @@ paths:
get:
tags:
- - Catalog API
+ - tables
+ - views
Review Comment:
Having `tables`/ `views` here simply means that if a catalog supports
`tables`**or** `views` then it needs to support this endpoint as you won't be
able to do something with tables if namespaces aren't supported. See also
https://github.com/apache/iceberg/pull/9940#issuecomment-2177235732.
Also I probably wouldn't tag any endpoint with `credentials-vending`, since
that capability doesn't require to implement certain endpoints.
--
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
