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

This decreases the confirm_req_batches_max from 20 to 2 #3148

Merged
merged 1 commit into from
Mar 16, 2021

Conversation

clemahieu
Copy link
Contributor

Equivalent to 280/sec -> 28/sec. This option is also made configurable.

Confirm_req batches are used in the bootstrap process and excessive requests to reps can degrade performance and often get dropped.

…o 280/sec -> 28/sec. This option is also made configurable.

Confirm_req batches are used in the bootstrap process and excessive requests to reps can degrade performance and often get dropped.
@zhyatt zhyatt added performance Performance/resource utilization improvement toml TOML related change labels Mar 15, 2021
@zhyatt zhyatt added this to the V22.0 milestone Mar 15, 2021
@zhyatt zhyatt requested a review from SergiySW March 15, 2021 20:58
@clemahieu clemahieu merged commit 0ffa688 into develop Mar 16, 2021
deusmax pushed a commit to deusmax/nano-node that referenced this pull request Mar 16, 2021
…o 280/sec -> 28/sec. This option is also made configurable. (nanocurrency#3148)

Confirm_req batches are used in the bootstrap process and excessive requests to reps can degrade performance and often get dropped.
@zhyatt zhyatt deleted the confirm_req_batches branch September 21, 2021 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance Performance/resource utilization improvement toml TOML related change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants