Skip to content
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

perf: use more rayon over tokio #983

Closed
wants to merge 14 commits into from
Closed

Conversation

baszalmstra
Copy link
Collaborator

This PR replaces our tokio logic for IO with rayon yielding a significant performance improvement at least on Windows. But we need more testing on other operating systems.

This also incorporates the changes from #940.

> ./target/release/rattler_main create --platform win-64 jupyterlab -c https://prefix.dev/conda-forge 
...
48sec 220ms 578µs

With these changes:

> timeit { ./target/release/rattler create --platform win-64 jupyterlab -c https://prefix.dev/conda-forge }
...
2sec 620ms 702µs

@baszalmstra baszalmstra changed the title perf: validate using threads perf: use more rayon over tokio Dec 13, 2024
@wolfv wolfv closed this Dec 16, 2024
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