Skip to content

[r3.4] db/state: fix MaxStep using MinTxNum in DomainRoTx.prune#20635

Merged
AskAlexSharov merged 1 commit into
release/3.4from
alex/fix_domain_maxstep_34
Apr 18, 2026
Merged

[r3.4] db/state: fix MaxStep using MinTxNum in DomainRoTx.prune#20635
AskAlexSharov merged 1 commit into
release/3.4from
alex/fix_domain_maxstep_34

Conversation

@AskAlexSharov

Copy link
Copy Markdown
Collaborator

Cherry-pick of #20609 to release/3.4

@AskAlexSharov AskAlexSharov enabled auto-merge (squash) April 18, 2026 01:43

@erigon-copilot erigon-copilot Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[SharovBot]

LGTM.

Clear correctness fix: was incorrectly derived from , duplicating and reporting the wrong upper bound for the pruned range. Switching it to matches the intended min/max semantics and the rest of the prune stats structure.

@AskAlexSharov AskAlexSharov merged commit 45aff2e into release/3.4 Apr 18, 2026
16 of 21 checks passed
@AskAlexSharov AskAlexSharov deleted the alex/fix_domain_maxstep_34 branch April 18, 2026 01:50
AskAlexSharov added a commit that referenced this pull request Apr 18, 2026
Cherry-pick of #20609 to release/3.4

Co-authored-by: bendertherobert <bendertherobert@gmail.com>
github-merge-queue Bot pushed a commit that referenced this pull request Apr 20, 2026
Cherry-pick from `release/3.4` to `main`:

- #20635 db/state: fix MaxStep using MinTxNum in DomainRoTx.prune
- #20639 dbg.SaveHeapProfileNearOOMPeriodically() call in all
sub-commands
- #20571 blk-range: wrong logs
- #20565 merge: set merge workers to 1
- #20576 simplify user-visible logs
- #20518 cp related PRs for "don't query in db, what is in files"
- #20517 teach: Unwind beyond data in snapshots not allowed
- #20368 rpc: use synctest for TestClientCancelHTTP to fix CI flakiness
- #20329 logs: move some Info logs to Debug level
- #20262 stageloop, execmodule: add ERIGON_IN_MEM_HISTORY env flag for
in-mem history reads

---------

Co-authored-by: bendertherobert <bendertherobert@gmail.com>
Co-authored-by: moskud <sudeepdino008@gmail.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: info@weblogix.biz <admin@10gbps.weblogix.it>
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