1
100%
master: 100%

Ran 28 Feb 2016 11:45PM UTC

Files 64

Run time 2s

Badge

Embed ▾

Committed 28 Feb 2016 10:55PM UTC coverage: 94.857% (-0.06%) from 94.918%

Job # TERM=dumb

Build Type

push

travis-ci

Committed by

ben-manes

Commit Message

Fix async refresh from prematurely replacing the value

When a refresh-after-write is triggered the entry should be reloaded in
the background, the present value available for reads, and atomically
replaced when the new value has been loaded. This operation is atomic
with other writes to that entry and is blocking (rather than clobbering)
if another write is attempted.

This should work the same for synchronous and asynchronous caches, but
unfortunately it wasn't. For an asycn cache a new, incomplete future
was immediately put into the cache and to be consumed by the next
request. Due to layering, the custom reloadAsync was not called and
the operation delegated to load instead. This was of course wrong and
not the intended (or expected) behavior, so it is now fixed.

Thanks to Etienne Houle @ Stingray for notifying me of this problem.

Coverage Stats

5035 of 5308 relevant lines covered (94.86%)

0.95 hits per line

ben-manes / caffeine / 1626 / 1
100%
master: 100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1626.1 (TERM=dumb)

ben-manes / caffeine / 1626 / 1 100% master: 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1626.1 (TERM=dumb)

ben-manes / caffeine / 1626 / 1
100%
master: 100%

README BADGES
x