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.

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

net processing: requeue transaction GETDATA requests more frequently #18955

Closed
wants to merge 4 commits into from

Conversation

jnewbery
Copy link
Contributor

Requeuing requests after the previous timeout means that if the transaction has cleared from g_already_asked_for, we'll wait up to a minute until we request it from another peer. Instead, set the process_time more frequently, so we'll rerequest it sooner.

jnewbery and others added 4 commits May 8, 2020 13:47
PoissonNextSend is no longer used only for net. Since it's
a utility random function, move it to random.cpp
Poisson distributions are used for more than just the next inv send.
Make the name more generic.

-BEGIN VERIFY SCRIPT-
sed -i 's/\<PoissonNextSend\>/GetPoissonRand/g' $(git grep -l 'PoissonNextSend' ./src)
-END VERIFY SCRIPT-
This means that if the transaction has cleared out from
g_already_asked_for, we'll rerequest from an announcing peer sooner.
@fanquake fanquake added the P2P label May 12, 2020
@jnewbery
Copy link
Contributor Author

Oops. p2p_txdownload.py still failing. I'll re-open once I've fixed it up.

@jnewbery jnewbery closed this May 12, 2020
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Feb 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants