Skip to content

Conversation

@AdamGS
Copy link
Contributor

@AdamGS AdamGS commented Jan 21, 2026

Experimenting showed that just using sccache is generally better or roughly equal, at least for our current setup with runs-on backed storage.
I've audited all of our CI setup, and found a few places where we run on github runners so using sccache is wasteful and disabled it, and also found a few places where rust-cache should still be useful.

@AdamGS AdamGS added the experiment Tests an idea label Jan 21, 2026
@AdamGS AdamGS changed the title Experiment with not using the rust-cache action Experiment with not using the rust-cache action for CI Jan 21, 2026
@codspeed-hq
Copy link

codspeed-hq bot commented Jan 21, 2026

Merging this PR will not alter performance

✅ 1254 untouched benchmarks
⏩ 1254 skipped benchmarks1


Comparing adamg/experiment-no-rust-cache (c5d2f92) with develop (59918a4)

Open in CodSpeed

Footnotes

  1. 1254 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@AdamGS AdamGS changed the title Experiment with not using the rust-cache action for CI Remove the rust-action based cache Jan 21, 2026
@AdamGS AdamGS added fix chore Release label indicating a trivial change and removed experiment Tests an idea fix labels Jan 21, 2026
Signed-off-by: Adam Gutglick <adam@spiraldb.com>
Signed-off-by: Adam Gutglick <adam@spiraldb.com>
Signed-off-by: Adam Gutglick <adam@spiraldb.com>
Signed-off-by: Adam Gutglick <adam@spiraldb.com>
@AdamGS AdamGS force-pushed the adamg/experiment-no-rust-cache branch from 98ee758 to c5d2f92 Compare January 21, 2026 13:48
@AdamGS AdamGS marked this pull request as ready for review January 21, 2026 13:48
@AdamGS AdamGS merged commit edeef00 into develop Jan 21, 2026
46 of 47 checks passed
@AdamGS AdamGS deleted the adamg/experiment-no-rust-cache branch January 21, 2026 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Release label indicating a trivial change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants