[ENG-35624]: Add azure storage support for metadata extractor#180
[ENG-35624]: Add azure storage support for metadata extractor#180dharmendersheshma merged 17 commits intomainfrom
Conversation
dharmendersheshma
left a comment
dharmendersheshma
left a comment
There was a problem hiding this comment.
/push-image ENG-35624-azure
dharmendersheshma
left a comment
dharmendersheshma
left a comment
There was a problem hiding this comment.
/push-image ENG-35624-azure-lat
dharmendersheshma
left a comment
dharmendersheshma
left a comment
There was a problem hiding this comment.
/push-image ENG-35624-azure-new
| validateAzureConfig(azureConfig); | ||
|
|
||
| BlobServiceClientBuilder builder = new BlobServiceClientBuilder(); | ||
| String endpoint = String.format("https://%s.blob.core.windows.net", azureConfig.getAccountName()); |
There was a problem hiding this comment.
can you use .dfs.core.windows.net endpoint ?
There was a problem hiding this comment.
dfs and blob are interchangeable and it should work fine if customer provides dfs as well. Anyway changed to dfs client now.
lakeview/src/main/java/ai/onehouse/config/models/common/AzureConfig.java
Show resolved
Hide resolved
dharmendersheshma
left a comment
dharmendersheshma
left a comment
There was a problem hiding this comment.
/push-image ENG-35624-azure-new-2
dharmendersheshma
left a comment
dharmendersheshma
left a comment
There was a problem hiding this comment.
/push-image ENG-35624-azure-new-3
dharmendersheshma
left a comment
dharmendersheshma
left a comment
There was a problem hiding this comment.
/push-image ENG-35624-azure-new-4
dharmendersheshma
left a comment
dharmendersheshma
left a comment
There was a problem hiding this comment.
/push-image ENG-35624-azure-new-5
dharmendersheshma
left a comment
dharmendersheshma
left a comment
There was a problem hiding this comment.
/push-image ENG-35624-azure-final
dharmendersheshma
left a comment
dharmendersheshma
left a comment
There was a problem hiding this comment.
/push-image ENG-35624
dharmendersheshma
left a comment
dharmendersheshma
left a comment
There was a problem hiding this comment.
/push-image ENG-35624-endgame
|
3 participants
Testing
Azure client successfully created and authenticated:
