stevenzwu commented on code in PR #15691:
URL: https://github.com/apache/iceberg/pull/15691#discussion_r2963548342


##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -2191,6 +2191,16 @@ components:
         type: string
       example: [ "accounting", "tax" ]
 
+    CatalogObjectType:
+      type: string
+      description:
+        The type of a named catalog object within a namespace. Identifiers 
MUST be unique
+        across all catalog object types within the same namespace; a table and 
a view with
+        the same name in the same namespace are not allowed. Known types are 
listed below.
+      enum:

Review Comment:
   in the future, new object types can be added like index.
   
   we can probably also add the `namespace` too. but there is no rename 
namespace support yet. so it isn't required at the moment. If we do add 
namespace as a catalog object type, the descriptions need to be adjusted.



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

Reply via email to