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


##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -980,6 +980,30 @@ paths:
           schema:
             type: string
             enum: [ all, refs ]
+        - in: query
+          name: referenced-by
+          description:
+            A comma-separated list of fully qualified view names (namespace 
and view name) representing the view
+            reference chain when a table is loaded via a view. The list should 
be ordered with the outermost view

Review Comment:
   Sorry, I mean more like a practice I would not encourage.
   
   I feel like if you have an invoker, that is a signal that the data behind 
the invoker is very sensitive and only certain users should have it. If you add 
a definer on top of the invoker view it's like you are bypassing that. Better 
to build a definer view in the first place



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