+1 on the spec change. It’s a solid first step toward enabling DEFINER views. As usual, the spec change is intentionally kept separate from access control.
Yufei On Wed, Sep 3, 2025 at 8:18 AM huaxin gao <[email protected]> wrote: > +1 (non-binding) > > On Tue, Sep 2, 2025 at 6:38 PM Prashant Singh <[email protected]> > wrote: > >> Hi All, >> I propose adding an *optional* referenced-by to the REST loadTable call, >> which will contain the fully qualified name of the view (namespace of the >> view name and the view name) in case the table is being referenced by a >> view. >> >> This will be really helpful in a couple of ways : >> 1. First step towards enabling DEFINER >> <https://trino.io/docs/current/sql/create-view.html#security> views >> 2. Audit, incase one wants to track what's the base objects accessed from >> the direct object accessed (example: doc >> <https://docs.snowflake.com/en/sql-reference/account-usage/access_history#columns>) >> >> >> For details please check: >> >> - Spec change PR: https://github.com/apache/iceberg/pull/13810 >> - Reference Implementation PR: >> https://github.com/apache/iceberg/pull/13979 >> - Discuss Thread: >> https://lists.apache.org/thread/01gb9rygdd1gqks7lnl1o6440qocnh9m >> >> Please vote in the next 72 hours: >> >> [ ] +1 Add these changes to the spec >> [ ] +0 >> [ ] -1 I have questions and/or concerns >> >> Best, >> Prashant Singh >> >
