Copilot commented on code in PR #8712:
URL: https://github.com/apache/gravitino/pull/8712#discussion_r2543898599
##########
core/src/main/java/org/apache/gravitino/cache/CaffeineEntityCache.java:
##########
@@ -176,7 +196,47 @@ public boolean invalidate(NameIdentifier ident,
Entity.EntityType type) {
checkArguments(ident, type);
return segmentedLock.withLock(
EntityCacheRelationKey.of(ident, type),
- () -> invalidateEntities(ident, type, Optional.empty()));
+ () -> {
+ // Clear possible relation first, then clear the main entity cache.
+ // For example, if a tag has been updated, apart from invalidating
the the relation:
Review Comment:
Typo: "the the relation" should be "the relation"
```suggestion
// For example, if a tag has been updated, apart from invalidating
the relation:
```
--
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]
