-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
experimental provider system #8070
experimental provider system #8070
Conversation
3fbb6aa
to
105b730
Compare
acbd82f
to
dbdee7f
Compare
3556ad2
to
476444c
Compare
dbdee7f
to
c5a49d2
Compare
c5a49d2
to
35c25f1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The way experiment is introduced LGTM, some small style tweaks inline.
ShortDescription: ` | ||
Returns statistics about the content the node is advertising. | ||
|
||
This interface is not stable and may change from release to release. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💭 (not related to this PR, but in general) would be cool if go-ipfs-cmds had a flag to indicate "experimental" and "deprecated" hint, so markdown generated by https://github.com/ipfs/http-api-docs could reflect that in some visual way.
Perhaps a simple boolean flag would do? How could I kick this off? PR against go-ipfs-cmds?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure, probably? Maybe start an issue in go-ipfs for tracking and collecting feedback before starting on the PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, with a couple very minor comments
d9d6ba3
to
2fd55d1
Compare
The batched provider system is enabled when the experimental AcceleratedDHTClient is enabled There is also an `ipfs stats provide` command which gives stats about the providing/reproviding system when the batched provider system is enabled
…T Client and batched providing system
e226399
to
acd4683
Compare
Adds an experimental provider system that makes use of the experimental dht client.
Depends on ipfs/go-ipfs-provider#34, links to #8061
The point of this system is to be able to quickly provide by being sympathetic to how Kademlia works. For the time being it tries to get records into the DHT very quickly at the expense of having a spike in resource usage during that time.
It also comes with a few extras that have been added for sanity purposes during testing/evaluation and that may be useful in production including: