yuqi1129 commented on code in PR #8922:
URL: https://github.com/apache/gravitino/pull/8922#discussion_r2472115826
##########
api/src/main/java/org/apache/gravitino/rel/Table.java:
##########
@@ -99,6 +99,33 @@ default Map<String, String> properties() {
return Collections.emptyMap();
}
+ /**
+ * Formats the table as a string representation.
+ *
+ * @return the formatted string representation of the table
+ */
+ default String format() {
+ throw new UnsupportedOperationException("Table format is not supported.");
+ }
+
+ /**
+ * Gets the location of the table.
+ *
+ * @return the location of the table
+ */
+ default String location() {
+ throw new UnsupportedOperationException("Table location is not
supported.");
+ }
+
+ /**
+ * Indicates whether the table is external.
+ *
+ * @return true if the table is external, false otherwise
+ */
Review Comment:
I have no preference on this, as we haven't used it yet, and the concept is
not so deterministic for a managed table. So adding or removing it is fine with
me.
--
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]
