-
Notifications
You must be signed in to change notification settings - Fork 295
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
Pintracker/local operation logic revamp #308
Comments
I am going to re-use this issue to describe what we need to do in order to improve local operation handling (that is, a pin/unpin/sync/recover requests that try to match the ipfs state to the shared state). This is logic taken care of by the There are a bunch of problems:
The pin tracker works OK in an scenario with small amount of pin requests that can be processed in time, but not with large sets of pins injected to cluster. The pin tracker implementation is relatively old and naive and needs to step up to current needs. |
This fixes #326. It adds a new `pin_method` configuration option to the `ipfshttp` component allows to configure it to perform `refs -r <cid>` before the `pin/add` call. By fetching content before pinning, we don't have a global lock in place, and we can have several pin-requests to ipfs in parallel. It also adds a `concurrent_pins` option to the pin tracker, which launches more pin workers so it can potentially trigger more pins at the same time. This is a minimal intervention in the pintracker as #308 is still pending. Documentation for the configuration file has been updated. License: MIT Signed-off-by: Hector Sanjuan <code@hector.link>
This fixes #326. It adds a new `pin_method` configuration option to the `ipfshttp` component allows to configure it to perform `refs -r <cid>` before the `pin/add` call. By fetching content before pinning, we don't have a global lock in place, and we can have several pin-requests to ipfs in parallel. It also adds a `concurrent_pins` option to the pin tracker, which launches more pin workers so it can potentially trigger more pins at the same time. This is a minimal intervention in the pintracker as #308 is still pending. Documentation for the configuration file has been updated. License: MIT Signed-off-by: Hector Sanjuan <code@hector.link>
This fixes #326. It adds a new `pin_method` configuration option to the `ipfshttp` component allows to configure it to perform `refs -r <cid>` before the `pin/add` call. By fetching content before pinning, we don't have a global lock in place, and we can have several pin-requests to ipfs in parallel. It also adds a `concurrent_pins` option to the pin tracker, which launches more pin workers so it can potentially trigger more pins at the same time. This is a minimal intervention in the pintracker as #308 is still pending. Documentation for the configuration file has been updated. License: MIT Signed-off-by: Hector Sanjuan <code@hector.link>
A post to ipfs will hang and doesn't time out unless ipfs itself does.
We need:
The text was updated successfully, but these errors were encountered: