Send multiple inv messages if mempool.size > MAX_INV_SZ #3153

pull gavinandresen wants to merge 1 commits into bitcoin:master from gavinandresen:multiple_inv changing 1 files +4 −2

gavinandresen commented at 4:17 AM on October 26, 2013: contributor

Bug-fix cherry-picked from the 'smart fee' work:

Changes the response to the 'mempool' command so that if the memory pool has more than MAX_INV_SZ transactions (50,000) it will respond with multiple 'inv' messages.

Send multiple inv messages if mempool.size > MAX_INV_SZ

Changes the response to the 'mempool' command so that if
the memory pool has more than MAX_INV_SZ transactions (50,000)
it will respond with multiple 'inv' messages.

1f3d3647ff

BitcoinPullTester commented at 4:34 AM on October 26, 2013: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/1f3d3647ffb49943f485c9a663f0644ca7d110de 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.
petertodd commented at 6:35 AM on October 26, 2013: contributor

ACK
laanwj commented at 7:33 AM on October 26, 2013: member

Looks like a sensible change, ACK
sipa referenced this in commit e213005177 on Oct 26, 2013
sipa merged this on Oct 26, 2013
sipa closed this on Oct 26, 2013
gavinandresen deleted the branch on Nov 4, 2013
Bushstar referenced this in commit b4e19f8dd5 on Apr 8, 2020
DrahtBot locked this on Sep 8, 2021

Contributors