jackye1995 commented on PR #6772: URL: https://github.com/apache/iceberg/pull/6772#issuecomment-1426873914
We have different places for creating file path names, - `LocationProvider` for data files - `TableOperations.metadataFileLocation` for manifests and manifest lists - catalog specific impl for metadata file location (e.g. `BaseMetastoreTableOperations.newTableMetadataFilePath`) So I will restrict this PR to just data location, and do other ones later to avoid having too many changes in different places. -- 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
