henrib commented on code in PR #13801:
URL: https://github.com/apache/iceberg/pull/13801#discussion_r2285166533


##########
api/src/main/java/org/apache/iceberg/catalog/Catalog.java:
##########
@@ -325,6 +326,47 @@ default boolean dropTable(TableIdentifier identifier) {
    */
   Table loadTable(TableIdentifier identifier);
 
+    /**
+     * Computes the eTag of a given table.
+     *
+     * <p>This fetches the up-to-date eTag for a given table identifier</p>
+     *
+     * @param identifier the table identifier
+     * @return the eTag as a type 5 uuid or null if the eTag could not be 
computed
+     */
+    default UUID getTableETag(TableIdentifier identifier) {

Review Comment:
   Hi @gaborkaszab,
   I was trying to address the comments and derive a way to compute an eTag 
from the metadata location. The original PR - and I will revert to this - is 
only meant to address from the HiveCatalog perspective a way to assess 
staleness of a Table object from a server side perspective. The REST catalog in 
Hive is embedded and to cache table objects, the initial issue and PR seems 
like a reasonable ask.
   Whether the same idea can be applied broadly or to other catalogs is 
certainly valid but at this point, it should probably be discussed in its own 
issue.
   



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

Reply via email to