search

apache / iceberg

Apache Iceberg
https://iceberg.apache.org/
Apache License 2.0
6.49k stars 2.24k forks source link

Support WASB scheme in ADLSFileIO #11504

Closed mrcnc closed 1 week ago

mrcnc commented 1 week ago

This is a second attempt to resolve https://github.com/apache/iceberg/issues/10127 but avoids using java.net.URI for parsing, which was found to be problematic after the first attempt was merged.

Additionally I've refactored to minimize the number of lines changed and clarify the usage of the storageAccount variable, which was previously storing the endpoint host and now will only store the storage account name as the subdomain.

bryanck commented 1 week ago

Thanks for the PR @mrcnc ! And for the reviews @RussellSpitzer @amogh-jahagirdar @jbonofre !