-
-
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
migrate to datastore-backed peerstore #6080
base: master
Are you sure you want to change the base?
Conversation
🎉 Awesome! 🎉
For now, we should probably add an experimental option to the config (https://github.com/ipfs/go-ipfs-config/blob/master/experiments.go). Concretely:
|
Makes sense, @Stebalien. Will look into those when I get some spare cycles. Thanks for the quick follow-up! |
License: MIT Signed-off-by: Raúl Kripalani <raul@protocol.ai>
@raulk i'm getting a panic:
|
@raulk could you rebase this on latest master? There are a few conflicts and i'd like to continue testing with some changes that landed in master. |
The datastore-backed libp2p peerstore is now production ready. Let's roll it out.
Could someone enumerate a few tests in go-ipfs land that would make intensive use of the peerstore? I'd like to benchmark memory and gc before we merge.
EDIT: do we want to make the peerstore choice and options configurable?