-
Notifications
You must be signed in to change notification settings - Fork 86
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
churn no timeouts fix #4959
churn no timeouts fix #4959
Conversation
66bd878
to
1295559
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well done.
let praosSyncTargets = min 1 (targetNumberOfActiveBigLedgerPeers base) | ||
in case regime of | ||
ChurnBootstrapPraosSync -> praosSync | ||
ChurnPraosSync -> praosSync | ||
ChurnBootstrapPraosSync -> praosSyncTargets | ||
ChurnPraosSync -> praosSyncTargets |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We use praosSync
in other places. Maybe change it in other places as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I renamed most of those to deadlineTargets as under Praos that's all we have and which is modified by stepping thru the churn loop. Here I decided to keep the praosSync prefix instead of renaming the let binding to deadlineTargets as maybe it would be slightly confusing.
@@ -414,7 +414,7 @@ peerChurnGovernor PeerChurnArgs { | |||
-> HotValency | |||
-> PeerSelectionTargets | |||
-> ModifyPeerSelectionTargets | |||
decreaseKnownBigLedgerPeers _ _ targets base = |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
well spotted
1295559
to
ae74431
Compare
Description
This was a pain to find.
resolves #4922
Checklist
Quality
Maintenance
ouroboros-network
project.