Re: [PR] REST spec: Add Optional UUID for Namespaces to OpenAPI spec [iceberg]

["danielcweeks (via GitHub)" <gi...@apache.org>]

danielcweeks commented on code in PR #15317:
URL: https://github.com/apache/iceberg/pull/15317#discussion_r2954384097


##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -2185,11 +2156,23 @@ components:
           $ref: '#/components/schemas/TableIdentifier'
 
     Namespace:
-      description: Reference to one or more levels of a namespace
-      type: array
-      items:
-        type: string
-      example: [ "accounting", "tax" ]
+      oneOf:

Review Comment:
   This looks like it would be an incompatible change.  We might need to consider a new endpoint to represent a new structure (e.g. a v2 list)



-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org