eric-maynard commented on code in PR #1983: URL: https://github.com/apache/polaris/pull/1983#discussion_r2176472162
########## site/content/in-dev/unreleased/access-control.md: ########## Review Comment: There are a few other issues in this doc, such as the fact that a Policy is now a valid securable object, and the note that `If you update the privileges bestowed to a service principal, the updates won't take effect for up to one hour.`. I also think the diagram at the bottom is confusing at best. However, this change by itself looks good enough to merge as it's just not accurate. -- 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]
