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

Reply via email to