singhpk234 commented on code in PR #13879:
URL: https://github.com/apache/iceberg/pull/13879#discussion_r2298822949
##########
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:
> May also be worth specifying that Identity(col) vs Col are either
equivalent
IMHO using either should be fine ? though i can't think of a case where i
would prefer one over the other. let me add clarification (they are equivalent)
statement.
> if the engine cannot evaluate any of the terms, it MUST NOT return the
column. If a term cannot be evaluated but it is not required for processing, it
may be ignored.
Agree, but should we ignore the column since we don't understand the
transform ? my understanding is we should fail the query, how about we say this
:
if the client cannot evaluate any of the terms, it MUST fail processing. If
a term cannot be evaluated but it is not required for processing, it may be
ignored.
--
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]
