rdblue commented on code in PR #9940:
URL: https://github.com/apache/iceberg/pull/9940#discussion_r1521825954


##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -61,6 +61,14 @@ security:
   - OAuth2: [catalog]
   - BearerAuth: []
 
+tags:
+  - name: views
+    description: Requires server to support views

Review Comment:
   I'm not sure I agree with this wording. Maybe something more like "Required 
as part of view support"?
   
   To me, the goal is to clarify what parts of this spec are optional and what 
are grouped together. Views are a great example: if a service supports views, 
it has to support everything tagged `views`.



-- 
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

Reply via email to