-
Notifications
You must be signed in to change notification settings - Fork 1.9k
perf(tree): worker pooling for storage in multiproof generation #18887
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 11 commits
Commits
Show all changes
53 commits
Select commit
Hold shift + click to select a range
436dbd7
feat(storage): introduce storage proof worker pool
yongkangc fbeec50
fmt, clippy
yongkangc ab82386
add fallback
yongkangc 13891ad
fix comments
yongkangc d4e0adb
refactor(metrics): remove unused storage proof metrics from ProofTask…
yongkangc 2957afa
refactor(proof_task): improve documentation and rename variables for …
yongkangc 800dcf6
refactor(proof_task): streamline documentation and clarify task manag…
yongkangc 29d48d4
refactor(config): remove storage proof worker configuration
yongkangc 3fb97c6
refactor(proof_task): enhance comments and adjust queue capacity logic
yongkangc 5779b86
disable max concurrency
yongkangc 0e33837
nits
yongkangc 3bcbc71
Update crates/trie/parallel/src/proof_task.rs
yongkangc 4a67076
Update crates/trie/parallel/src/proof_task.rs
yongkangc b2d5bcc
using unbounded queue
yongkangc 8f4e3a1
rm comment
yongkangc b4bf193
refactor(proof_task): optimize storage proof computation by reusing c…
yongkangc 6282d2e
propogate error up
yongkangc 838dc67
reduce scope of pr - exclude all accs
yongkangc 5897945
fmt, clippy
yongkangc 6b5de7c
fmt
yongkangc 05e0eb8
refactor(proof_task): consolidate blinded storage node with storage p…
yongkangc 4829de9
rm comment
yongkangc 6472cfe
simplify worker concurrency
yongkangc 61ecc9a
bump to error!
yongkangc 30f6fda
Update crates/engine/tree/src/tree/payload_processor/mod.rs
yongkangc 4680336
handle sending error back
yongkangc 58d6f8b
fmt
yongkangc 59b0353
fix fmt
yongkangc 93c67e8
Enhance TreeConfig with storage worker count configuration
yongkangc 1954502
update message
yongkangc 2429320
refactor(proof_task): use impl bound
yongkangc 1902b43
make spawning falliable
yongkangc 8fb0dd1
remove error log, as we propogate up
yongkangc e0010d7
Apply suggestion from @Copilot
yongkangc 53cd4ba
use expect instead of unwrap
yongkangc e854628
Update crates/trie/parallel/src/proof_task.rs
yongkangc 3b17cc7
Update crates/trie/parallel/src/proof_task.rs
yongkangc 6c89cf4
consolidate
yongkangc c5f6eb9
removed the unnecessary remaining_concurrency variable allocation
yongkangc af73c7a
clippy
yongkangc a8e52bc
Apply suggestion from @yongkangc
yongkangc c48b328
address brian's pr
yongkangc 3eff3e2
Merge branch 'main' into yk/worker_pool_storage_2
yongkangc 7efff3d
Update crates/trie/parallel/src/proof_task.rs
yongkangc f7cd93f
Update crates/trie/parallel/src/proof_task.rs
yongkangc f823c6b
Refactor error handling in StorageWorkerJob to use a consistent error…
yongkangc 9c08aed
Refactor error handling in StorageWorkerJob to use structured error t…
yongkangc 7f9ec06
fmt, clipy
yongkangc 6d41352
fix
yongkangc 4ca404e
bump up workers
yongkangc d66ed61
cli flag for storage
yongkangc 4aba3de
docs: update CLI reference for storage-worker-count flag
yongkangc 8e64738
docs: clarify storage-worker-count uses Tokio blocking pool
yongkangc File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.