Skip to content
This repository has been archived by the owner on Dec 7, 2022. It is now read-only.

[Feature Request] Azure Storage #68

Open
CodingDive opened this issue Nov 27, 2018 · 4 comments
Open

[Feature Request] Azure Storage #68

CodingDive opened this issue Nov 27, 2018 · 4 comments

Comments

@CodingDive
Copy link

Hey, I would love to have support for azure storage. The Azure team recently published a new library https://github.com/Azure/azure-storage-js and it would be great to have this integrated as an alternative to AWS S3.

@OJezu
Copy link
Contributor

OJezu commented Dec 6, 2019

Are PRs accepted?

@RomainLanz
Copy link
Member

Yes it is!

@OJezu
Copy link
Contributor

OJezu commented Dec 6, 2019

@RomainLanz append method for Azure works only for special "Append Blobs", while for most of the time, more usual "Block Blobs" are used. Should I make it, that:

  1. append for azure creates "Append Blobs"
  2. append is not implemented
  3. append manipulates Block list of Block Blobs and appends to those - this will take some time to implement, and requires 3 API calls AFAIK

Personally, as this is a abstraction lib, I would avoid using different types of blobs at all, and leave append unimplemented for now.

@RomainLanz
Copy link
Member

RomainLanz commented Dec 7, 2019

If you believe it's better not to mix different types of blobs, go for it. I'm not an expert on Azure.
We can always change that later if we need to.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants