fivetran-arunsuri commented on PR #2197: URL: https://github.com/apache/polaris/pull/2197#issuecomment-3229076860
Thanks for the review @dimas-b @eric-maynard I’ll update the changelog as suggested. On the feature flag — since this API is already restricted to root, I’m not sure a flag adds much additional protection. If you feel it does, could you clarify how you envision feature-flagging this change? It feels like privilege-based access (as you mentioned) would be the more scalable long-term solution, so that specific roles can be granted reset access instead of relying on a global on/off switch. Happy to add a toggle if you feel strongly, but my preference would be to keep the API available and focus on evolving the privilege model as followup PR -- 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: issues-unsubscr...@polaris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
