jerqi commented on code in PR #4055:
URL: https://github.com/apache/gravitino/pull/4055#discussion_r1768236068
##########
core/src/main/java/org/apache/gravitino/storage/relational/RelationalEntityStore.java:
##########
@@ -88,8 +89,12 @@ public void setSerDe(EntitySerDe entitySerDe) {
@Override
public <E extends Entity & HasIdentifier> List<E> list(
- Namespace namespace, Class<E> type, Entity.EntityType entityType) throws
IOException {
- return backend.list(namespace, entityType);
+ Namespace namespace,
+ Class<E> type,
+ Entity.EntityType entityType,
+ List<Field> allowMissingFields)
Review Comment:
Yes, because different upper layers may ignore different fields. For
example, list roles
Some upper layers may ignore objects and objects count.
Some upper layers may only ignore objects.
Some upper layers may not ignore any field.
--
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]
