Skip to content

wire --erigondb.domain.steps-in-frozen-file for stage_exec and seg retire#21025

Merged
sudeepdino008 merged 2 commits into
mainfrom
extract-erigondb-domain-flag
May 7, 2026
Merged

wire --erigondb.domain.steps-in-frozen-file for stage_exec and seg retire#21025
sudeepdino008 merged 2 commits into
mainfrom
extract-erigondb-domain-flag

Conversation

@sudeepdino008

Copy link
Copy Markdown
Member

Summary

Wire the existing --erigondb.domain.steps-in-frozen-file flag into two more entry points so the domain-only merge cap can be overridden when running these tools directly:

  • integration stage_exec — registers the flag and applies it via AggOpts.ErigondbDomainStepsInFrozenFile when constructing the aggregator singleton.
  • erigon seg retire — registers the flag on the seg retire subcommand and applies it via a new Aggregator.SetErigondbDomainStepsInFrozenFile runtime setter (the aggregator is constructed via openAgg here, so the override is applied post-construction, before any merge work begins).

History/inverted-index merges are unaffected — the flag still only adjusts the domain merge cap, matching the main erigon binary's behavior.

Both commits cherry-picked from the explores_3 exploration branch.

Test plan

  • make erigon integration builds clean
  • make lint reports 0 issues
  • Manual smoke: run integration stage_exec --erigondb.domain.steps-in-frozen-file=Inf and erigon seg retire --erigondb.domain.steps-in-frozen-file=64 against a test datadir, confirm the override-log line fires and merges respect the new cap

@sudeepdino008 sudeepdino008 changed the title cmd/integration, cmd/utils/app: wire --erigondb.domain.steps-in-frozen-file for stage_exec and seg retire wire --erigondb.domain.steps-in-frozen-file for stage_exec and seg retire May 7, 2026
@AskAlexSharov AskAlexSharov added this pull request to the merge queue May 7, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 7, 2026
@sudeepdino008 sudeepdino008 added this pull request to the merge queue May 7, 2026
Merged via the queue into main with commit ec7a361 May 7, 2026
38 checks passed
@sudeepdino008 sudeepdino008 deleted the extract-erigondb-domain-flag branch May 7, 2026 09:03
AskAlexSharov pushed a commit that referenced this pull request May 7, 2026
Cherry-pick of #20930, #21025, and #21026 onto performance.

---------

Co-authored-by: sudeepdino008 <sudeepdino008@gmail.com>
AskAlexSharov pushed a commit that referenced this pull request May 13, 2026
….steps-in-frozen-file (#21148)

Cherry-pick of #20705 to `release/3.4`.

Conflict-resolution notes:
- Kept `stepSize`/`stepsInFrozenFile` as `uint64` (release/3.4 hasn't
picked up the `atomic.Uint64` refactor from main); only the new
`erigondbDomainStepsInFrozenFile` override field was added.
- Did not pull in unrelated test helpers
(`filledDomainWithHashMapAccessor`,
`TestDomain_KeyPosResetOnCollisionRetry`) that exist on main but not on
`release/3.4`.
- `node/eth/backend.go`: kept the `db/consensuschain` import and added
`db/config3` alongside it.

Prerequisite for cherry-pick of #21025 to `release/3.4`.
AskAlexSharov pushed a commit that referenced this pull request May 13, 2026
… and `seg retire` (#21149)

Requires #21148 to be merged before that one.


Cherry-pick of #21025 to `release/3.4`.

---------

Co-authored-by: moskud <sudeepdino008@gmail.com>
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