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

Add batch processing to improve performance #10

Merged
merged 2 commits into from
Nov 17, 2015

Conversation

dignifiedquire
Copy link
Contributor

In reference to my comment here, ipfs/notes#2 (comment) I tried to add some parallelized batch processing which has improved resource usage and performance drastically for me.

cc @bengl

@davglass
Copy link
Owner

@dignifiedquire Looks like you forgot to add the async module to the package.json so the build failed. Can you add that and make sure that all of the current tests pass? Maybe even adding in more? Thanks! 🍻

@dignifiedquire
Copy link
Contributor Author

@davglass uups sorry about that :( should be all good now. The current tests are passing now, not sure how to add a good test for this though. If you have any suggestions I'm happy to write one.

@dignifiedquire dignifiedquire mentioned this pull request Nov 16, 2015
53 tasks
@RichardLitt RichardLitt mentioned this pull request Nov 16, 2015
42 tasks
@daviddias
Copy link

Been using this PR and it is golden! :D

@davglass
Copy link
Owner

I'll let @bengl handle this since he's been running this and registry-static for me for a while. It's his call 😄

@bengl bengl merged commit f71a2d8 into davglass:master Nov 17, 2015
@dignifiedquire dignifiedquire deleted the parallel branch March 25, 2016 20:58
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.

4 participants