Skip to content

Azure Storage: Azure storage support #3253#3254

Merged
sebastian-alfers merged 81 commits into
akka:mainfrom
sfali:3253_azure_storage
Oct 3, 2024
Merged

Azure Storage: Azure storage support #3253#3254
sebastian-alfers merged 81 commits into
akka:mainfrom
sfali:3253_azure_storage

Conversation

@sfali

@sfali sfali commented Aug 27, 2024

Copy link
Copy Markdown
Contributor

References #3253

Work in progress, still needs to add more tests and documentations.

Code style is copied from AWS S3 connector.

sfali added 30 commits August 23, 2024 16:55
It parses the returned XML and populate StorageException
1. Populates required date and version header
2. Reads current request and create authorization token as per AWS requirements
3. Populates authorization header
Current implementation only supports Blob and File services.
Currently supported functions:
1. Get blob and file
2. Put blob
3. Create file
3. Put range
4. Clear range
5. Delete blob and file
6. Get blob and file properties
1. Create query string with SAS token
2. Initialize query string
sfali and others added 3 commits September 9, 2024 11:03
…/StorageAttributes.scala

Co-authored-by: Shubham Girdhar <girdharshubham@hotmail.com>
…/impl/AzureStorageStream.scala

Co-authored-by: Shubham Girdhar <girdharshubham@hotmail.com>
…/models.scala

Co-authored-by: Shubham Girdhar <girdharshubham@hotmail.com>

@girdharshubham girdharshubham left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good so far.

sfali and others added 6 commits September 10, 2024 10:45
…/StorageAttributes.scala

Co-authored-by: Shubham Girdhar <girdharshubham@hotmail.com>
…/StorageAttributes.scala

Co-authored-by: Shubham Girdhar <girdharshubham@hotmail.com>
1. Make sure we don't emit empty string
2. correct configuration
1. Request builder for delete container
2. Implementation of delete container and Scala and Java API
3. Tests for delete container
4. Documentation
1. Request builder for create & delete directories
2. Implementation of create & delete directories and Scala and Java API
3. Tests for create & delete directories
4. Documentation
@sfali

sfali commented Sep 14, 2024

Copy link
Copy Markdown
Contributor Author

Issue with Java API was misconfiguration, not sure why AzuriteIntegrationSpec is failing, any idea @girdharshubham?

@girdharshubham

Copy link
Copy Markdown
Contributor

Issue with Java API was misconfiguration, not sure why AzuriteIntegrationSpec is failing, any idea @girdharshubham?

Hi @sfali
Sorry this got delayed, I will check this soon.

@sebastian-alfers

Copy link
Copy Markdown
Contributor

Hi @sfali, can you share and update on this PR? Do you want to see your contribution merged for the next release?

@sfali

sfali commented Sep 30, 2024

Copy link
Copy Markdown
Contributor Author

Hi @sfali, can you share and update on this PR? Do you want to see your contribution merged for the next release?

What do you want me to share @sebastian-alfers? The PR is failing due to couchbase and ironmq connectors are failing. not sure what should I do with these connectors.

@sebastian-alfers

Copy link
Copy Markdown
Contributor

We fixed both in main, so feel free to rebase and give it a try.

@sfali

sfali commented Sep 30, 2024

Copy link
Copy Markdown
Contributor Author

We fixed both in main, so feel free to rebase and give it a try.

@sebastian-alfers Now scaaldoc and jms connector failed. Can pin point what's going? Can you please take a look and let me know what's going on.

Thanks

@sebastian-alfers

Copy link
Copy Markdown
Contributor

Thank you for bringing this to our attention. The scaladoc failure can be ignored for now, and the jms was something we just fixed in main, so please rebase again.

@sfali

sfali commented Oct 1, 2024

Copy link
Copy Markdown
Contributor Author

Thank you for bringing this to our attention. The scaladoc failure can be ignored for now, and the jms was something we just fixed in main, so please rebase again.

Thanks @sebastian-alfers , Everything is passing other than scaaldoc

@sebastian-alfers

Copy link
Copy Markdown
Contributor

Please let me know once you want to merge it.

@sfali

sfali commented Oct 2, 2024

Copy link
Copy Markdown
Contributor Author

Please let me know once you want to merge it.

This can be merge anytime. It needs to be approved.

@sfali

sfali commented Oct 3, 2024

Copy link
Copy Markdown
Contributor Author

It seems scaladoc pipeline needs to be successful before it can be merged. @sebastian-alfers

@sebastian-alfers sebastian-alfers merged commit 4049dca into akka:main Oct 3, 2024
@sfali sfali deleted the 3253_azure_storage branch October 3, 2024 17:37
objectPath = objectPath,
requestBuilder = requestBuilder)

private[storage] def putBlob(objectPath: String,

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sfali one question for large files may be of > 5-10Gb .. S3 has multipartUploadAPI for azure blob how are we going to achieve it ?

splitting in chunks?

adding some UTs would have been helpful..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependency-change For PRs changing the version of a dependency. documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants