rdblue commented on code in PR #13879:
URL: https://github.com/apache/iceberg/pull/13879#discussion_r2294428381
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -3260,6 +3260,33 @@ components:
additionalProperties:
type: string
+ ReadRestrictions:
+ type: object
+ description: >
+ Read Restrictions for a table including projection and row filter
expressions.
+ The client MUST enforce these rules to read data from the table.
+ If the read-restrictions section is not present or is empty, clients
MUST treat it as equivalent to having no restrictions.
+ properties:
+ required-projection:
+ description: >
+ A list of projections that must be applied before query
projections. If the term is a transform, it must replace the column referenced
by the term.
+ For example, if the term is mask(cc, 0, 4) i.e mask transform on
column cc, it must replace the column cc in the query with the masked value,
Review Comment:
I don't think that this should use a `mask` example unless that transform is
defined. The example I was giving was also a credit card where the first 12
characters are masked (hence the `cc` column name). Changing the example to
make it slightly shorter loses the connection to a real-world case that readers
are familiar with.
--
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]
