turborepo
Include timeSaved metric when skipping cache check
#4952
Merged

Include timeSaved metric when skipping cache check #4952

mehulkar
turbo-orchestrator turbo-orchestrator added team: turborepo
vercel
github-actions
mehulkar mehulkar force pushed from d92094a1 to d380677d 2 years ago
mehulkar mehulkar force pushed from d380677d to bcd49d65 2 years ago
mehulkar mehulkar changed the title Include timeSaved metric when Include timeSaved metric when skipping cache check 2 years ago
refactor: Change Itemstatus signature to include TimeSaved metric
f9f3e166
mehulkar Improve log message when cache restoration is skipped
57c05e02
mehulkar Return timeSaved metric even when cache restoration is skipped
01a4a6de
Fix how Exists() works to make clear that its not composite, but firs…
c39306c9
mehulkar mehulkar force pushed from bcd49d65 to c39306c9 2 years ago
mehulkar
mehulkar commented on 2023-05-20
Exit the loop early on a cache hit, since we're propaging a cache hit…
b1dd3bc7
Check for a cahce hit after handling errors
73674e82
extract duration extracter function instead of just a const
92bb2a4f
Make method private
405bbbf3
comments
5a450083
mehulkar
mehulkar commented on 2023-05-20
mehulkar
mehulkar commented on 2023-05-20
mehulkar mehulkar marked this pull request as ready for review 2 years ago
mehulkar mehulkar requested a review 2 years ago
mehulkar mehulkar requested a review from tknickman tknickman 2 years ago
mehulkar mehulkar requested a review from chris-olszewski chris-olszewski 2 years ago
mehulkar mehulkar requested a review from gsoltis gsoltis 2 years ago
gsoltis gsoltis requested a review from NicholasLYang NicholasLYang 2 years ago
NicholasLYang
NicholasLYang approved these changes on 2023-05-22
mehulkar
mehulkar commented on 2023-05-23
mehulkar Update cli/internal/runcache/runcache.go
b08e291e
Use const for cache source noop
a7984592
maschwenk
maschwenk commented on 2023-05-23
chris-olszewski
chris-olszewski commented on 2023-05-25
Use CacheMiss helper for noop implementation
320a3684
mehulkar mehulkar force pushed from 5f3c73fe to 320a3684 2 years ago
mehulkar Merge branch 'main' into mehulkar/turbo-1000-runs-include-timesaved-f…
46fc3f48
mehulkar
mehulkar Store and retrieve timeSaved value from daemon (#5101)
2f1b9fd3
mehulkar Remove debug statements
d94af444
mehulkar Fix linter
8ea8c8ad
mehulkar mehulkar requested a review from chris-olszewski chris-olszewski 2 years ago
mehulkar mehulkar requested a review from NicholasLYang NicholasLYang 2 years ago
mehulkar Merge branch 'main' into mehulkar/turbo-1000-runs-include-timesaved-f…
ea267a67
mehulkar revert log message
55105bc7
mehulkar Remove Exists() check
32296b80
mehulkar set cache source to fs when skipping restoration
5bc4f123
tknickman
tknickman approved these changes on 2023-05-30
mehulkar Revert all the cacheStatus signature things
f5e81833
mehulkar Revert the parts of runcache that aren't related to daemon get/set
e96c3bb2
mehulkar
mehulkar mehulkar merged d5897fdc into main 2 years ago
mehulkar mehulkar deleted the mehulkar/turbo-1000-runs-include-timesaved-for-local-runs branch 2 years ago

Login to write a write a comment.

Login via GitHub

Assignees
No one assigned
Labels
Milestone