xunliu commented on code in PR #4690:
URL: https://github.com/apache/gravitino/pull/4690#discussion_r1746360390
##########
core/src/main/java/org/apache/gravitino/meta/RoleEntity.java:
##########
@@ -91,16 +95,7 @@ public Map<String, String> properties() {
*/
@Override
public List<SecurableObject> securableObjects() {
- // The securable object is a special kind of entities. Some entity types
aren't the securable
- // object, such as
- // User, Role, etc.
- // The securable object identifier must be unique.
- // Gravitino assumes that the identifiers of the entities may be the same
if they have different
- // types.
- // So one type of them can't be the securable object at least if there are
the two same
- // identifier
- // entities .
Review Comment:
We don't need these comments any more?
--
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]
