mchades commented on code in PR #7973:
URL: https://github.com/apache/gravitino/pull/7973#discussion_r2272975003
##########
core/src/main/java/org/apache/gravitino/cache/EntityCacheKey.java:
##########
@@ -30,6 +30,24 @@ public class EntityCacheKey {
private final NameIdentifier identifier;
private final Entity.EntityType type;
private final SupportsRelationOperations.Type relationType;
+ private final Boolean allFields;
+
+ /**
+ * Creates a new instance of {@link EntityCacheKey} with the given arguments.
+ *
+ * @param ident The identifier of the entity.
+ * @param type The type of the entity.
+ * @param relationType The type of the relation, it can be null.
+ * @param allFields Whether to retrieve all fields of the entity.
+ * @return A new instance of {@link EntityCacheKey}.
+ */
+ public static EntityCacheKey of(
+ NameIdentifier ident,
+ Entity.EntityType type,
+ SupportsRelationOperations.Type relationType,
+ boolean allFields) {
+ return new EntityCacheKey(ident, type, relationType, allFields);
Review Comment:
why not cache when `allFields==true`?
--
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]
