sungwy commented on code in PR #13810:
URL: https://github.com/apache/iceberg/pull/13810#discussion_r2293754655


##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -960,6 +960,15 @@ paths:
           schema:
             type: string
             enum: [ all, refs ]
+        - in: query
+          name: referenced-by
+          description:
+            The fully qualified name of the view (i.e namespace name and view 
name) that referenced the table.
+            
+            nested namespaces are separated by a unit separator (`0x1F`) byte.

Review Comment:
   I think it would be helpful to adopt the requirements keyword from 
[RFC2119](https://datatracker.ietf.org/doc/html/rfc2119). This helps ensure 
there’s no ambiguity in how trust establishment influences authorization with 
this query parameter:
   
   ```suggestion
               nested namespaces are separated by a unit separator (`0x1F`) 
byte.
               
              Servers MUST ONLY use this parameter to influence authorization 
when the request is 
              authenticated as a Trusted Engine identity as determined by the 
catalog’s trust policy.
   ```



-- 
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]

Reply via email to