henrib commented on issue #13800:
URL: https://github.com/apache/iceberg/issues/13800#issuecomment-3193445420
Although this may be useful to other catalog implementations, it should not
be a mandatory method - as a method of the Catalog interface. Adding a default
method implementation that performs
client.loadTable().property("metadata_location") has very low usefulness as a a
one-liner that goes against the functional intent (fast access to last known
metadata location). And other catalog implementations might have different and
more efficient ways of assessing staleness. From that perspective, it does not
look like a good addition to the Catalog API.
And if it is determined later that this should be generalized, the signature
'public String getTableMetadataLocation(TableIdentifier identifier)' can always
be added to the Catalog interface. In the current state, I believe
https://github.com/apache/iceberg/pull/13801 should remain specific to
HiveCatalog.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
