laskoviymishka commented on code in PR #15750:
URL: https://github.com/apache/iceberg/pull/15750#discussion_r3656868599
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -4033,6 +4074,8 @@ components:
type: object
additionalProperties:
type: string
+ labels:
+ $ref: '#/components/schemas/Labels'
Review Comment:
Views can carry inherited labels (from the underlying tables / namespace),
and surfacing that on the view has real value: a consumer reading the view sees
the classification/context without re-deriving it from the sources.
The view use case came up multiple times during the sync discussions and in
the design doc.
It's a different call than ReadRestrictions because labels are informational
exposure, not an access decision that has to be enforced consistently, so the
reason to drop views there doesn't carry over one-to-one. The most important
use case is informational exchange for the reader: for example, added AI
semantic context.
--
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]
