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