Skip to content

integrity: blk/sec fix#20373

Merged
AskAlexSharov merged 3 commits intorelease/3.4from
alex/blk_sec_fix_34
Apr 7, 2026
Merged

integrity: blk/sec fix#20373
AskAlexSharov merged 3 commits intorelease/3.4from
alex/blk_sec_fix_34

Conversation

@AskAlexSharov
Copy link
Copy Markdown
Collaborator

No description provided.

@AskAlexSharov AskAlexSharov enabled auto-merge (squash) April 7, 2026 06:25
Copy link
Copy Markdown
Contributor

@erigon-copilot erigon-copilot bot left a comment

Choose a reason for hiding this comment

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

[SharovBot]

LGTM.

The previous field was effectively measuring windows completed per interval only indirectly when the sampler/checking work is chunked by window, which made the log misleading. Splitting it into:

  • from
  • from

is the right fix and makes the progress report unambiguous. The / bookkeeping is clean and local to the ticker goroutine.

@AskAlexSharov AskAlexSharov merged commit dc0e48e into release/3.4 Apr 7, 2026
16 of 22 checks passed
@AskAlexSharov AskAlexSharov deleted the alex/blk_sec_fix_34 branch April 7, 2026 06:51
AskAlexSharov added a commit that referenced this pull request Apr 8, 2026
github-merge-queue bot pushed a commit that referenced this pull request Apr 10, 2026
## Summary
Cherry-pick of release/3.4 PRs:
- #20368 rpc: use synctest for TestClientCancelHTTP to fix CI flakiness
- #20373 integrity: blk/sec fix
- #20372 execution/state: use index-based range over log topics to avoid
copy
- #20367 fix: preserve stored chain config when genesis hash doesn't
match known chain
- #20374 ParseVersion: optimize
- #20369 batchSize: more precise estimate
- #20399 hive: pin more things

---------

Co-authored-by: erigon-copilot[bot] <265817861+erigon-copilot[bot]@users.noreply.github.com>
Co-authored-by: root <giulio.rebuffo@gmail.com>
Co-authored-by: yperbasis <andrey.ashikhmin@gmail.com>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-authored-by: Muzry <muzrry@gmail.com>
Co-authored-by: muzry.li <muzry.li1@ambergroup.io>
Co-authored-by: Andrew Ashikhmin <34320705+yperbasis@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.

1 participant