flyrain commented on code in PR #16144:
URL: https://github.com/apache/iceberg/pull/16144#discussion_r3229986521
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -2267,6 +2267,17 @@ components:
type: string
nullable: false
+ CatalogObjectIdentifier:
+ description:
+ Reference to a catalog object (table, view, or namespace) as an
+ ordered list of hierarchical levels.
+ The object kind is determined by context (e.g. the endpoint or a
+ companion type discriminator), not by the identifier structure alone.
+ type: array
+ items:
+ type: string
Review Comment:
Do we apply any constraints on the table/view/namespace name? For example,
no slash(/) is allowed. Given we didn't specify any constraint on the table
identifier, it's not a blocker for this PR. We can work on that as a followup.
We can also discuss whether we could avoid any constraints in IRC, and relying
on the implementations(catalogs, engines) to cast their options.
--
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]
