RussellSpitzer commented on code in PR #13879:
URL: https://github.com/apache/iceberg/pull/13879#discussion_r2298758463
##########
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,
+ essentially projecting it as mask(cc, 0, 4) AS cc.
+ Readers are NOT allowed to project columns that are not listed and
must apply transforms.
+ If the required-projection is not present or is empty, it means
that no projection is required and all columns can be read as-is.
+
+ Note: That each column must have only a single projection, meaning
a column can be projected as-is or as a transformed value, but not both.
Review Comment:
```suggestion
Note: Each column may have only a single projection, meaning a
column can be projected as-is or as a transformed value, but not both.
```
Small typos in the beginning, but the second half of this I think is a
little confusing. You also can't have multiple projections that are transforms
as well. For example Truncate(5) and Truncate(3).
--
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]
