Skip to content

chore: remove unused rayon pool from WorkloadExecutor#19065

Merged
mattsse merged 3 commits intoparadigmxyz:mainfrom
0xKarl98:cleanup_WorkloadExecutor
Oct 16, 2025
Merged

chore: remove unused rayon pool from WorkloadExecutor#19065
mattsse merged 3 commits intoparadigmxyz:mainfrom
0xKarl98:cleanup_WorkloadExecutor

Conversation

@0xKarl98
Copy link
Contributor

@0xKarl98 0xKarl98 commented Oct 16, 2025

This closes issue #19064

cc @yongkangc

Copy link
Collaborator

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

lgtm

pending @yongkangc

cc @klkvr

@github-project-automation github-project-automation bot moved this from Backlog to In Progress in Reth Tracker Oct 16, 2025
@sashass1315
Copy link
Contributor

sashass1315 commented Oct 16, 2025

lgtm

pending @yongkangc

cc @klkvr

There is #18859 was before issue even, should i close that then? (it's not a problem i understand that my pr was before issue, so i can close it so there wasn't 2 same prs)

@mattsse
Copy link
Collaborator

mattsse commented Oct 16, 2025

ah sigh, sorry I missed this one,

I think we can combine both so that you're credited for this as well

@mattsse mattsse enabled auto-merge October 16, 2025 11:54
@mattsse mattsse added this pull request to the merge queue Oct 16, 2025
Merged via the queue into paradigmxyz:main with commit 7e006d6 Oct 16, 2025
41 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Reth Tracker Oct 16, 2025
emhane pushed a commit to op-rs/op-reth that referenced this pull request Oct 20, 2025
)

Co-authored-by: sashass1315 <sashass1315@gmail.com>
Co-authored-by: Matthias Seitz <matthias.seitz@outlook.de>
emhane pushed a commit to op-rs/op-reth that referenced this pull request Oct 20, 2025
)

Co-authored-by: sashass1315 <sashass1315@gmail.com>
Co-authored-by: Matthias Seitz <matthias.seitz@outlook.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants