plusplusjiajia commented on PR #758: URL: https://github.com/apache/paimon-rust/pull/758#issuecomment-5779257309
> This PR touches too many parts of the code; it adds a lot of public methods rather than simply modifying the implementation, so it's hard for me to judge whether the approach is reasonable. @JingsongLi Fair. The latest push trims it to 19 files: no new public method (the rest is `pub(crate)`), the `assume_authorized` flags, the `FileSystemCatalog` and bundled-server changes and the strict auth-response parsing are gone, and so is the live re-check of `query-auth.enabled` your two P1s hardened — the handle now trusts the option it was loaded with, as Java does, so an ordinary table costs nothing. What remains: the grant is fetched at planning and rides on the split. -- 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]
