tengqm commented on code in PR #6698:
URL: https://github.com/apache/gravitino/pull/6698#discussion_r2002220339
##########
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:
Can you help provide some rationale behind this change?
--
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]
