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

parallelize upload to chunk with setting "chunkParallelize": true #876

Closed
wants to merge 7 commits into from

Conversation

simonegosetto
Copy link

I think it's an important feature for upload performance, in my test this is a 400% more faster then the "serial" chunks upload

@simonegosetto
Copy link
Author

@Akash4686 yes, this is the goal of this PR

# Conflicts:
#	dist/filepond.esm.min.js
#	dist/filepond.min.js
@rikschennink
Copy link
Collaborator

Appreciate the PR, currently working on v5, not sure yet if I can find time to merge this.

@simonegosetto
Copy link
Author

@rikschennink I understand perfectly, but I recommend making an effort because it could be a feature that improves the sight of many, other libraries already have this feature

@rikschennink
Copy link
Collaborator

It's queued for me to further take a look at 👍

@rikschennink
Copy link
Collaborator

Hi @simonegosetto, just wanted to let you know that I've been pushing this forward for the past six months. At this time mainly because that way I don't forget to add it to v5. Just so you know that it's still on my radar.

@simonegosetto
Copy link
Author

hi @rikschennink, very good thank you, let me know if I can do anything to help you.

@rikschennink
Copy link
Collaborator

I've added this feature to v5, it can now upload chunks in parallel.

@simonegosetto
Copy link
Author

very good news

@simonegosetto
Copy link
Author

when v5 will available in npm?

@rikschennink
Copy link
Collaborator

I really really really want to get an alpha version out there before December

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

Successfully merging this pull request may close these issues.

2 participants