omerhadari opened a new issue, #941:
URL: https://github.com/apache/iceberg-rust/issues/941

   The 
[warehouse](https://github.com/apache/iceberg-rust/blob/c754e27f752a9e2f86180a9b40f69a3d4ab2dae9/crates/catalog/glue/src/catalog.rs#L46)
 config prop for Glue catalog is required, but it isn't always known or 
required. The Glue catalog ID is enough in order to resolve metadata locations.
   
   Moreover - I don't think this path is used for anything but protocol (FileIO 
type) inference? I was able to work around this issue by passing the correct 
Glue catalog ID (which is the AWS account ID) and `s3://` as the warehouse, 
with no issue.
   
   In my opinion `warehouse` should be replaced with something like `FileIO` 
type and only the Glue catalog ID should be required, but for backwards 
compatibility not passing warehouse should at least be allowed. 


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