amogh-jahagirdar commented on code in PR #8800:
URL: https://github.com/apache/iceberg/pull/8800#discussion_r1355713680
##########
api/src/main/java/org/apache/iceberg/Table.java:
##########
@@ -333,6 +333,15 @@ default UpdateStatistics updateStatistics() {
*/
Map<String, SnapshotRef> refs();
+ /**
+ * Returns the UUID of the table
+ *
+ * @return the UUID of the table
+ */
+ default String uuid() {
Review Comment:
That's a good point, we don't need to stick to the internal types on the
API. If I think about what's best here from an API perspective, it would be to
surface a UUID type.
From a simple API semantic perspective, for someone unfamiliar with all of
this, it's probably just makes more sense that `table.uuid()` returns an actual
java.util.UUID.
Also if one wants to store this data in external systems and leverage their
UUID data types (which would be more compact/efficient for lookups, although
probably negligible in practice) then the API avoids the extra conversion
burden on the user. I think this advantage is probably minimal/negligible
though.
--
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: issues-unsubscr...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
