Limit number of known addresses per peer #4257

pull sipa wants to merge 1 commits into bitcoin:master from sipa:limitpeers changing 2 files +3 −2

sipa commented at 10:02 AM on May 30, 2014: member

No description provided.
Limit number of known addresses per peer 5823449e2d
Diapolo commented at 10:17 AM on May 30, 2014: none

Why is this important, what does it achieve? Missing a clear commit message :).
laanwj commented at 10:20 AM on May 30, 2014: member

@diapolo This limits memory usage per node
BitcoinPullTester commented at 11:18 AM on May 30, 2014: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/5823449e2d7f76fa1f24fed881a8a5041b34b6bf for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.
gavinandresen commented at 2:14 PM on May 30, 2014: contributor

ACK
laanwj commented at 2:21 PM on May 30, 2014: member

ACK on code changes, currently running this on my node to see if any problems appear.
gmaxwell commented at 7:02 PM on May 31, 2014: contributor

ACK
laanwj merged this on Jun 1, 2014
laanwj closed this on Jun 1, 2014
laanwj referenced this in commit cc342a3ede on Jun 1, 2014
MarcoFalke locked this on Sep 8, 2021

Contributors