tejanshrana opened a new pull request, #8911:
URL: https://github.com/apache/paimon/pull/8911

   ### Problem
   catalogDatabasePath() rejected any database whose location does not end in 
'.db' before checking the storage location, so 
metadata.iceberg.storage-location =table-location which writes Iceberg metadata 
beside the table and does not need the warehouse layout, was also rejected for 
databases with a non-warehouse location (e.g. an externally-provisioned or 
cross-account catalog database).
   
   ### Fix
   Move the '.db' guard into the catalog-storage branch, which is the only path 
that derives a warehouse-relative iceberg/<db>/ location by stripping the '.db' 
suffix. table-location now returns the database path as-is with no layout 
requirement; catalog-storage still requires '.db' and gives a clearer message.
   
   ### Tests
   Adds unit tests covering a non-warehouse database under both storage 
locations.
   


-- 
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]

Reply via email to