c-thiel commented on code in PR #12584:
URL: https://github.com/apache/iceberg/pull/12584#discussion_r2586004519
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -2010,11 +2065,34 @@ components:
Namespace:
description: Reference to one or more levels of a namespace
+ allOf:
+ - $ref: '#/components/schemas/CatalogObject'
+ example: [ "accounting", "tax" ]
+
+ CatalogObject:
Review Comment:
Makes sense! I'll update the PR later today.
Regarding the question on yet another identifier: We had this discussion in
one of the Catalog syncs, I used the existing identifiers in an earlier version.
The main motivation here is that clients should not need to know what type
an object is and would require additional requests to detect this. This is
undesired, especially as this would need to be done before each request, as the
type of an object might change (drop view, create table with same name).
--
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]
