Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow Azure govcloud for native Azure file system #21307

Closed
mosabua opened this issue Mar 28, 2024 · 0 comments · Fixed by #23071
Closed

Allow Azure govcloud for native Azure file system #21307

mosabua opened this issue Mar 28, 2024 · 0 comments · Fixed by #23071

Comments

@mosabua
Copy link
Member

mosabua commented Mar 28, 2024

https://trinodb.slack.com/archives/C0305TQ05KL/p1711623700672519

Azure gov cloud uses https://%s.blob.core.usgovcloudapi.net not https://%s.blob.core.windows.net ,

The validation does not allow this and needs to be changed to be more flexible.

private static final String INVALID_LOCATION_MESSAGE = "Invalid Azure location. Expected form is 'abfs://[<containerName>@]<accountName>.dfs.core.windows.net/<filePath>': %s";

private static final String INVALID_LOCATION_MESSAGE = "Invalid Azure location. Expected form is 'abfs://[@].dfs.core.windows.net/': %s";

@findepi findepi changed the title Alllow Azure govcloud for native Azure file system Allow Azure govcloud for native Azure file system Mar 28, 2024
@anusudarsan anusudarsan assigned anusudarsan and unassigned electrum May 21, 2024
@anusudarsan anusudarsan removed their assignment Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging a pull request may close this issue.

3 participants