jerqi commented on code in PR #9029:
URL: https://github.com/apache/gravitino/pull/9029#discussion_r2497698848


##########
api/src/main/java/org/apache/gravitino/rel/Table.java:
##########
@@ -44,6 +44,28 @@
 @Evolving
 public interface Table extends Auditable {
 
+  /**
+   * The reserved property name for the table format. This property indicates 
the format of the
+   * table, such as "iceberg", "hudi", "lance", etc. Some catalog 
implementations may use this
+   * property to determine the table format and perform specific operations 
based on the format.
+   */
+  String PROPERTY_TABLE_FORMAT = "format";

Review Comment:
   Do we need more constraints? This is a enum, isn't this? If this a Hive 
table,   is this a hive?  For jdbc, is this a jdbc?



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