geruh commented on code in PR #1642: URL: https://github.com/apache/iceberg-python/pull/1642#discussion_r1955110870
########## pyiceberg/table/locations.py: ########## @@ -64,6 +71,35 @@ def new_data_location(self, data_file_name: str, partition_key: Optional[Partiti str: A fully-qualified location URI for the data file. """ + def new_table_metadata_file_location(self, new_version: int = 0) -> str: Review Comment: @smaheshwar-pltr Regarding point 1, I had the same question while working on this. Centralizing locations/paths would definitely make future changes easier, especially with the ongoing discussions around relative path work. I found a related issue in the main repo here: https://github.com/apache/iceberg/issues/6809. Which looks like it didn't get any traction but still holds some value. -- 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