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

Simplify vote generator #4748

Merged
merged 3 commits into from
Oct 22, 2024
Merged

Conversation

RickiNano
Copy link
Contributor

@RickiNano RickiNano commented Oct 8, 2024

This PR removes the vote_generator_threshold that generates an additional wait to bundle more votes into each message. That wait period adds voting latency.
The logic that handles processing of requests and candidates have also been simplified. There was also a bug that request would only process when candidates were empty.
Bob has performance tested this branch and found no performance improvement nor degredatation
screenshot_2024-10-08_at_08 18 29

An interesting side effect is that the amount of active election timeouts dropped with this branch
screenshot_2024-10-08_at_08 21 18

Here is some testing on the NanoTicker node on live network. The screenshots are taken right after the daily saturation test.
image

@RickiNano RickiNano marked this pull request as ready for review October 11, 2024 10:21
@gr0vity-dev-bot
Copy link

gr0vity-dev-bot commented Oct 18, 2024

Test Results for Commit dd08220

Pull Request 4748: Results
Overall Status:

Test Case Results

  • 5n4pr_conf_10k_bintree: PASS (Duration: 109s)
  • 5n4pr_conf_10k_change: PASS (Duration: 212s)
  • 5n4pr_conf_change_dependant: PASS (Duration: 156s)
  • 5n4pr_conf_change_independant: PASS (Duration: 156s)
  • 5n4pr_conf_send_dependant: PASS (Duration: 157s)
  • 5n4pr_conf_send_independant: PASS (Duration: 138s)
  • 5n4pr_rocks_10k_bintree: PASS (Duration: 106s)
  • 5n4pr_rocks_10k_change: PASS (Duration: 216s)

Last updated: 2024-10-21 15:34:18 UTC

@qwahzi qwahzi added this to the V28 milestone Oct 19, 2024
@pwojcikdev pwojcikdev merged commit 4995718 into nanocurrency:develop Oct 22, 2024
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Merged / V28.0
Development

Successfully merging this pull request may close these issues.

4 participants