roryqi commented on PR #12482: URL: https://github.com/apache/gravitino/pull/12482#issuecomment-5364251708
> > LGTM, no other comments, wait for CI. > > The CI failure is caused by listTables using PROBE_TABLE_LIKE_AUTHORIZATION_EXPRESSION, while the new expression missed the existing owner semantics. I'll update PROBE_TABLE_LIKE_AUTHORIZATION_EXPRESSION to keep METALAKE/CATALOG/SCHEMA/TABLE owner access, so listTables preserves the previous behavior while still using the new probe privilege expression. You can use different expression constants. If we add owner check, it will add more checks, it will cost more time. You can define two constants one is for loading table, the other is for listing table. The listing table constant contains the owner check. The loading table constant doesn't contain the owner check. -- 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]
