Skip to content

Bump max ledger notifications value#4930

Merged
pwojcikdev merged 1 commit intonanocurrency:developfrom
pwojcikdev:bump-max-ledger-notifications
Jul 14, 2025
Merged

Bump max ledger notifications value#4930
pwojcikdev merged 1 commit intonanocurrency:developfrom
pwojcikdev:bump-max-ledger-notifications

Conversation

@pwojcikdev
Copy link
Copy Markdown
Contributor

During bootstrapping it's expected that there will be around 300 notifications queued per second, as measured on my node. This low default value was causing excessive warnings.

@gr0vity-dev-bot
Copy link
Copy Markdown

gr0vity-dev-bot commented Jul 14, 2025

Test Results for Commit 0515d0e

Pull Request 4930: Results
Overall Status:

Test Case Results

  • 5n4pr_conf_10k_change: PASS (Duration: 133s)
  • 5n4pr_conf_change_dependant: PASS (Duration: 135s)
  • 5n4pr_conf_change_independant: PASS (Duration: 131s)
  • 5n4pr_conf_send_dependant: PASS (Duration: 126s)
  • 5n4pr_conf_send_independant: PASS (Duration: 131s)
  • 5n4pr_rocks_10k_bintree: PASS (Duration: 117s)
  • 5n4pr_rocks_10k_change: PASS (Duration: 163s)

Last updated: 2025-07-14 19:31:56 UTC

@pwojcikdev pwojcikdev merged commit 98c1cdc into nanocurrency:develop Jul 14, 2025
26 checks passed
pwojcikdev added a commit that referenced this pull request Jul 14, 2025
@pwojcikdev pwojcikdev deleted the bump-max-ledger-notifications branch July 14, 2025 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants