jerqi commented on code in PR #6698: URL: https://github.com/apache/gravitino/pull/6698#discussion_r2005203858
########## authorizations/authorization-common/src/main/java/org/apache/gravitino/authorization/common/PathBasedMetadataObject.java: ########## @@ -31,7 +31,16 @@ public class PathBasedMetadataObject implements AuthorizationMetadataObject { */ public enum Type implements AuthorizationMetadataObject.Type { /** A path is mapped the path of storages like HDFS, S3 etc. */ - PATH(MetadataObject.Type.FILESET); + FILESET_PATH(MetadataObject.Type.FILESET), + /** A path is mapped the path of table storage like Hive. */ + TABLE_PATH(MetadataObject.Type.TABLE), + /** A path is mapped the path of schema storage like Hive. */ + SCHEMA_PATH(MetadataObject.Type.SCHEMA), + /** A path is mapped the path of cluster storage like Hive. */ + CATALOG_PATH(MetadataObject.Type.CATALOG), + /** A path is mapped the path of all cluster storages like Hive. */ + METALAKE_PATH(MetadataObject.Type.METALAKE); Review Comment: I will. Maybe I should give you background. How does a Hive table store? First, we will store the metadata in the MySQL, And then we will store the data in the HDFS location. For Hive, there is a default location for the cluster level. We can also specify other locations for schema and table levels. So a path won't only be fileset. -- 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: commits-unsubscr...@gravitino.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org