mchades commented on code in PR #7610:
URL: https://github.com/apache/gravitino/pull/7610#discussion_r2259462593


##########
api/src/main/java/org/apache/gravitino/rel/expressions/NamedReference.java:
##########
@@ -105,4 +116,46 @@ public String toString() {
       return String.join(".", fieldName);
     }
   }
+
+  /** A {@link NamedReference} that references a metadata field. */
+  final class MetadataField implements NamedReference {
+    /**
+     * The name of the metadata field that contains the partition name. This 
is used to reference
+     * the partition name in metadata fields.
+     */
+    public static final String PARTITION_NAME_FIELD = "partition_name";

Review Comment:
   suggest using `public static final MetadataField PARTITION_NAME_FIELD =xxx`, 
so that users can use it directly



-- 
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]

Reply via email to