Skip to content

chore: avoid hammering store at ntx builder startup#2064

Open
SantiagoPittella wants to merge 3 commits intomainfrom
santiagopittella-avoid-ntx-builder-spam-main
Open

chore: avoid hammering store at ntx builder startup#2064
SantiagoPittella wants to merge 3 commits intomainfrom
santiagopittella-avoid-ntx-builder-spam-main

Conversation

@SantiagoPittella
Copy link
Copy Markdown
Collaborator

@SantiagoPittella SantiagoPittella commented May 8, 2026

closes #1952

This PR:

  • Trusts local committed account state on restart and only fetches the unconsumed_notes delta per known account, dropping the per-account GetNetworkAccountDetailsById call.
  • Adds a configurable hydration semaphore in StoreClient (defaulted to 4) that bounds concurrent in-flight startup RPCs.
  • Adds a separate store_sync_checkpoint column on chain_state (required a new SQL migration), distinct from the mempool-driven block_num.

I tried it by:

  1. initializing the main version of the node.
  2. Started multiple network monitors to create network accounts.
  3. Stoped everything.
  4. Installed and initialized the new version with the existing data (i.e. I didn't re-run bootstrap), which ran the migration
  5. Everything continue working as expected.

@SantiagoPittella SantiagoPittella changed the title chore: avoid hamming store at ntx builder startup chore: avoid hammering store at ntx builder startup May 8, 2026
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.

1 participant