1
72%
master: 72%

Ran 19 Jan 2024 09:11AM UTC

Files 655

Run time 13s

Badge

Embed ▾

Committed 19 Jan 2024 08:49AM UTC coverage: 62.641% (-0.06%) from 62.697%

Job # 018d20e9-56f9-4f4d-92f6-a6e61407071c.1

Build Type

push

buildkite

Committed by

web-flow

Commit Message

Addressed sync issue in workflow cache (#5605)

What changed?
We now use the value returned from PutIfExists, this is thread safe, either the new value is inserted and returned, or the existing value is returned.
We also do not need to pin values in the cache, if they get evicted while we use them that is fine.
Added some more error handling, the error handling should never be triggered though.

Why?
The old implementation had a race condition in creating the value and inserting it

How did you test it?

Potential risks
There could still be concurrency issues

Release notes

Documentation Changes

Run Details

91614 of 146252 relevant lines covered (62.64%)

2335.05 hits per line

uber / cadence / 018d20e9-56f9-4f4d-92f6-a6e61407071c / 1
72%
master: 72%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 018d20e9-56f9-4f4d-92f6-a6e61407071c.1

uber / cadence / 018d20e9-56f9-4f4d-92f6-a6e61407071c / 1 72% master: 72%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 018d20e9-56f9-4f4d-92f6-a6e61407071c.1

uber / cadence / 018d20e9-56f9-4f4d-92f6-a6e61407071c / 1
72%
master: 72%

README BADGES
x