Make `uv cache clean` parallel process safe
#15888Comparing
konsti/safe-cache-clean
(a9df4a5
) withmain
(fb71b07
)+3%
- Untouched
- 3
Benchmarks
Passed
Commits
Click on a commit to change the comparison rangeBase
main
fb71b07
+2.69%
Make `uv cache clean` concurrency safe
Currently, `uv cache clean` and `uv cache prune` can cause crashes in other uv processes running in parallel by removing their in-use files.
We can solve this by using a shared (read) lock on the cache directory, while the `uv cache` operations use an exclusive (write) lock. The drawback is that this is always one extra lock, and that we assume that all platforms support shared locks.
Once Rust 1.89 fulfills our N-2 policy, we can add support for these methods in fs_err and switch to https://doc.rust-lang.org/std/fs/struct.File.html#platform-specific-behavior-2.
a9df4a5
5 hours ago
by konstin