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

Cache chunk store #2095

Merged
merged 1 commit into from
Jun 30, 2021
Merged

Cache chunk store #2095

merged 1 commit into from
Jun 30, 2021

Conversation

jhiesey
Copy link
Contributor

@jhiesey jhiesey commented Jun 3, 2021

What is the purpose of this pull request? (put an "X" next to item)

[ ] Documentation update
[ ] Bug fix
[x] New feature
[ ] Other, please explain:

What changes did you make? (Give an overview)

Which issue (if any) does this pull request address?
#2080

Is there anything you'd like reviewers to focus on?

@jhiesey jhiesey marked this pull request as ready for review June 23, 2021 22:42
Copy link
Member

@alxhotel alxhotel left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@feross feross left a comment

Choose a reason for hiding this comment

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

This is nice work! Should really help speed things up quite a bit, as we've seen on Wormhole.app. Let's merge!

@jhiesey jhiesey merged commit d540058 into webtorrent:master Jun 30, 2021
webtorrent-bot pushed a commit that referenced this pull request Jun 30, 2021
# [1.1.0](v1.0.4...v1.1.0) (2021-06-30)

### Features

* Use a cache on the chunk store ([#2095](#2095)) ([d540058](d540058))
@webtorrent-bot
Copy link
Collaborator

🎉 This PR is included in version 1.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add CacheChunkStore around all chunk stores
5 participants