Skip to content

paged writer: lost amount of workers#20335

Merged
AskAlexSharov merged 1 commit intorelease/3.4from
alex/lost_workers_num_34
Apr 5, 2026
Merged

paged writer: lost amount of workers#20335
AskAlexSharov merged 1 commit intorelease/3.4from
alex/lost_workers_num_34

Conversation

@AskAlexSharov
Copy link
Copy Markdown
Collaborator

No description provided.

@AskAlexSharov AskAlexSharov enabled auto-merge (squash) April 5, 2026 07:52
@AskAlexSharov AskAlexSharov requested a review from JkLondon April 5, 2026 13:47
Copy link
Copy Markdown
Collaborator

@Giulio2002 Giulio2002 left a comment

Choose a reason for hiding this comment

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

LGTM — obvious one-line fix to pass the configured compressor worker count into NewPagedWriter instead of hardcoding 1.

@AskAlexSharov AskAlexSharov merged commit 8e2e20e into release/3.4 Apr 5, 2026
22 checks passed
@AskAlexSharov AskAlexSharov deleted the alex/lost_workers_num_34 branch April 5, 2026 18:07
github-merge-queue Bot pushed a commit that referenced this pull request Apr 6, 2026
Cherry-pick from release/3.4 to main.

Original PRs:
- #20335 — paged writer: lost amount of workers
- #20281 — pagedWriter and compressor to use same COMPRESS_WORKERS
variable
- #20262 — stageloop, execmodule: add ERIGON_IN_MEM_HISTORY env flag
- #20114 — crypto: zero-alloc, syncpool-free method to hash []byte type
- #20194 — execution/state: revert CodeSizePath in codeChange journal
entry

20 candidates checked total; 15 were already effectively on main.

---------

Co-authored-by: moskud <sudeepdino008@gmail.com>
Co-authored-by: awskii <awskii@users.noreply.github.com>
Co-authored-by: Giulio rebuffo <giulio.rebuffo@gmail.com>
Co-authored-by: lystopad <oleksandr.lystopad@erigon.tech>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.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