[GitHub] [iceberg] rdblue commented on a diff in pull request #4428: Core, API: Add getting refs and snapshot by ref to the Table API

[GitBox <gi...@apache.org>]

rdblue commented on code in PR #4428:
URL: https://github.com/apache/iceberg/pull/4428#discussion_r884916051


##########
api/src/main/java/org/apache/iceberg/Table.java:
##########
@@ -297,4 +297,25 @@ default AppendFiles newFastAppend() {
    * Returns a {@link LocationProvider} to provide locations for new data files.
    */
   LocationProvider locationProvider();
+
+  /**
+   * Returns the current refs for the table
+   *
+   * @return the current refs for the table
+   */
+  Map<String, SnapshotRef> refs();

Review Comment:
   Shouldn't this also have a default that creates a map of "main" to the current snapshot ID?



-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org