danielcweeks commented on code in PR #18203:
URL: https://github.com/apache/iceberg/pull/18203#discussion_r4074338393
##########
api/src/main/java/org/apache/iceberg/Table.java:
##########
@@ -335,6 +335,16 @@ default UpdatePartitionStatistics
updatePartitionStatistics() {
/** Returns a {@link FileIO} to read and write table data and metadata
files. */
FileIO io();
+ /**
+ * Returns the location of the current table metadata file.
+ *
+ * @return the current table metadata file location
+ */
+ default String metadataFileLocation() {
Review Comment:
I don't think we want to add this to the public API. It's likely that we
will remove the requirement for a physical metadata location, so adding this
now doesn't make sense. I'd rather implementations have to check/cast to
BaseTable to access the physical location.
--
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]
