Skip to content

feat(config): system memory bound default compiler concurrency limit#3478

Merged
CHr15F0x merged 7 commits into
mainfrom
chris/new-compiler-concurrency-limit
Jun 22, 2026
Merged

feat(config): system memory bound default compiler concurrency limit#3478
CHr15F0x merged 7 commits into
mainfrom
chris/new-compiler-concurrency-limit

Conversation

@CHr15F0x

Copy link
Copy Markdown
Contributor

The formula is:

Concurrency_Limit = min(floor((System_RAM - Margin) / Compiler_RAM_Limit), n_CPUs)

Margin is configurable and defaults to 4GiB.
Compiler_RAM_Limit is configurable and defaults to 4GiB.

If computed Concurrency_Limit turns out to be 0 and the rpc server is enabled, the process will exit with an error.

@CHr15F0x CHr15F0x marked this pull request as ready for review June 17, 2026 11:20
@CHr15F0x CHr15F0x requested a review from a team as a code owner June 17, 2026 11:20
@t00ts

t00ts commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Worth a CHANGELOG entry? Seems like we're changing that default value.

@sistemd sistemd 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.

LGTM

Comment thread crates/pathfinder/src/bin/pathfinder/main.rs Outdated
@CHr15F0x

Copy link
Copy Markdown
Contributor Author

Worth a CHANGELOG entry? Seems like we're changing that default value.

@t00ts good catch, thanks:

bbbde16

@CHr15F0x CHr15F0x force-pushed the chris/new-compiler-concurrency-limit branch from bbbde16 to 0154721 Compare June 22, 2026 11:15
@CHr15F0x CHr15F0x merged commit 72021ca into main Jun 22, 2026
12 checks passed
@CHr15F0x CHr15F0x deleted the chris/new-compiler-concurrency-limit branch June 22, 2026 11:52
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.

5 participants