Skip to content

[cargo-bazel] Avoid repeatedly hitting the filesystem during CrateIndexLookup#3589

Merged
illicitonion merged 3 commits intobazelbuild:mainfrom
hermeticbuild:zbarsky/sparse-index
Sep 4, 2025
Merged

[cargo-bazel] Avoid repeatedly hitting the filesystem during CrateIndexLookup#3589
illicitonion merged 3 commits intobazelbuild:mainfrom
hermeticbuild:zbarsky/sparse-index

Conversation

@dzbarsky
Copy link
Copy Markdown
Contributor

@dzbarsky dzbarsky commented Sep 2, 2025

Only use the crate index if we don't have the checksums already from the Cargo.lock. Also we can compute the crate_index just once.

@dzbarsky dzbarsky force-pushed the zbarsky/sparse-index branch 2 times, most recently from b32351c to 1b23656 Compare September 2, 2025 19:44
@dzbarsky dzbarsky force-pushed the zbarsky/sparse-index branch from 1b23656 to c01440a Compare September 2, 2025 22:33
Copy link
Copy Markdown
Collaborator

@illicitonion illicitonion left a comment

Choose a reason for hiding this comment

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

Nice, thank you!

@illicitonion illicitonion added this pull request to the merge queue Sep 4, 2025
Merged via the queue into bazelbuild:main with commit 0f8b797 Sep 4, 2025
3 checks passed
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