jerryshao commented on code in PR #7600:
URL: https://github.com/apache/gravitino/pull/7600#discussion_r2193871274
##########
core/src/main/java/org/apache/gravitino/Entity.java:
##########
@@ -58,40 +58,27 @@ public interface Entity extends Serializable {
/** Enumeration defining the types of entities in the Gravitino framework. */
@Getter
enum EntityType {
- METALAKE("ml", 0),
- CATALOG("ca", 1),
- SCHEMA("sc", 2),
- TABLE("ta", 3),
- COLUMN("co", 4),
- FILESET("fi", 5),
- TOPIC("to", 6),
- USER("us", 7),
- GROUP("gr", 8),
- ROLE("ro", 9),
- TAG("ta", 10),
- MODEL("mo", 11),
- MODEL_VERSION("mv", 12),
-
- AUDIT("au", 65534);
-
- // Short name can be used to identify the entity type in the logs,
persistent storage, etc.
- private final String shortName;
+ METALAKE(0),
+ CATALOG(1),
+ SCHEMA(2),
+ TABLE(3),
+ COLUMN(4),
+ FILESET(5),
+ TOPIC(6),
+ USER(7),
+ GROUP(8),
+ ROLE(9),
+ TAG(10),
+ MODEL(11),
+ MODEL_VERSION(12),
+ AUDIT(65534);
Review Comment:
I think the index number in enum is also not used anymore, and can be
removed.
--
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]
