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.

Sign up for GitHub

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

Jump to bottom

move go-libp2p-peerstore to p2p/host/peerstore #1667

Merged
merged 384 commits into from
Aug 15, 2022
Merged

move go-libp2p-peerstore to p2p/host/peerstore #1667

merged 384 commits into from
Aug 15, 2022

Conversation

marten-seemann
Copy link
Contributor

Part of #1556.

raulk and others added 30 commits September 4, 2018 18:07
@raulk
rename package mem => pstoremem.
d20a98c
@raulk
rename AddrMgr => AddrBook elsewhere.
7f38aaf
@raulk
rename NewPeerstoreWith(...) => NewPeerstore(...); adjust godoc.
6959b23
@raulk
increase wait times in tests.
2818d65
@raulk
add mafmt import alias.
65adc76
@raulk
Merge pull request #30 from raulk/refactor
ad9ccb9 
* Refactor the peerstore into interface, in-memory implementation, datastore implementation. 
* Create common test suites to test all implementations against. 
* Normalise naming: rename AddrMgr => AddrBook everywhere. 
* Create interface PeerMetadata, with in-memory implementation.
* Adjust wait times in time-dependent tests.
@raulk
add tests for keybook; []peer.ID => peer.IDSlice.
888f631
@raulk
optimise benchmark code.
9229870
@raulk
nested test cases in addr_book suite.
cd0cdec
@raulk
migrate Addrs() and PeersWithAddrs() to use txns, improve perf, bench…
8abb8a3 
…marks.
@raulk
regroup test utils; remove unused function.
9a52623
@raulk
normalise import aliases.
55afe7b
@raulk
Merge pull request #32 from raulk/keybook-tests
1e12471 
* Introduce tests for KeyBook to increase coverage.
* Change usages of []peer.ID to IDSlice, which supports sorting.
* Restructure AddrBook tests into subtests.
* Import aliases.
@raulk
add baseline benchmarks.
ea75b18
@raulk
change cache layout, now peer.ID => []ma.Multiaddr.
5c6b53e
@raulk
check cache in ClearAddrs and perform targeted delete.
de70a75
@raulk
Merge branch 'master' into txndatastore
32aa55e
@raulk
fix bug in PeersWithAddrs().
6b61869
@raulk
small fixes.
22da141
@raulk
increase test coverage.
78f457d
@raulk
Resolve TODO.
71e08d9
@raulk
address review comments.
202fdc2
@raulk
hopefully fix intermittent time-dependent test failure.
4433afc
@raulk
attempt to fix time-dependent tests in CI.
b9b3939
@raulk
migrate from bespoke TTL manager to db-managed TTLs.
80894be
@raulk
remove deprecated ttlManager.
db643f3
@raulk
performance optimisation round.
e38ee5c
@raulk
do not prettify cache keys (peer ids).
18e0eeb
@raulk
Merge branch 'txndatastore' into ttldatastore
7ebb753
@raulk
adjust ttl tests as badger doesn't support sub-second ttl.
57ae658
marten-seemann and others added 15 commits December 14, 2021 19:04
@marten-seemann
implement a clean shutdown for the memory address book
58c639a
@marten-seemann
Merge pull request #192 from libp2p/clean-shutdown
b40a117 
implement a clean shutdown for the memory address book
@marten-seemann
Merge pull request #191 from libp2p/various-addr-book-optimizations
2da8c8d 
optimize allocations in the memory address book
@marten-seemann
move AddrList to pstoremen, unexport it
aedfd63
@marten-seemann
Merge pull request #193 from libp2p/move-sorting
f9a486a 
move AddrList to pstoremen, unexport it
@marten-seemann
remove all calls to peer.ID.Validate
ead3cdd
@marten-seemann
remove the addr package
0466498
@marten-seemann
Merge pull request #195 from libp2p/remove-addr-package
7195c20 
remove the addr package
@marten-seemann
Merge pull request #194 from libp2p/remove-validate
abc52a3 
remove all calls to peer.ID.Validate
@mrd0ll4r
pstoremem: fix slice preallocation in GetProtocols (#198)
8a547c5
@MarcoPolo
Use a clock interface to better support testing
832b0bd
@MarcoPolo
Merge pull request #199 from libp2p/marco/with-clock
1da4a8e 
feat: use a clock interface to better support testing for pstoremem
@MarcoPolo
Use mock clock in pstoreds and tests
c2f25ed
@MarcoPolo
Merge pull request #200 from libp2p/marco/with-clock-2
7108bf5 
feat: Use a clock interface in pstoreds as well
@marten-seemann
stop using the peer.Set (#201)
29bf33f
@marten-seemann
Copy link
Contributor Author

We should fix libp2p/go-libp2p-peerstore#203 and libp2p/go-libp2p-peerstore#204 first. Tests are flaky.

@marten-seemann marten-seemann force-pushed the merge-peerstore branch 2 times, most recently from 786beee to 8db02ca Compare August 15, 2022 09:11
@marten-seemann marten-seemann merged commit 357ebfd into master Aug 15, 2022
@marten-seemann marten-seemann deleted the merge-peerstore branch November 8, 2022 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vyzo vyzo vyzo approved these changes

@Stebalien Stebalien Stebalien approved these changes

@MarcoPolo MarcoPolo Awaiting requested review from MarcoPolo

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.