1
100%
master: 100%

Ran 14 Mar 2022 06:09AM UTC

Files 74

Run time 4s

Badge

Embed ▾

Committed 14 Mar 2022 03:33AM UTC coverage: 94.161% (+0.09%) from 94.076%

Job # #2838.1

Build Type

push

github-actions

Committed by

ben-manes

Commit Message

Remove unneccesary top-level cache fields

Previously the cache implemented AbstractMap for the its convenient
putAll, equals, hashCode, and toString implementations. This came at
the cost of its private, unused keySet and values fields. Those Object
methods iterated over the entrySet, which returns a copy of the entry
to avoid instabiliy due to concurrent mutations. This changes to
internal iterators which avoids this unnecessary garbage pressure.

Equality is a complex subject and the implementation's internal docs
now better clarify the semantics. Due to the cache potentially holding
entries that are pending removal (expiration, reference collection),
we expect that usages call Cache.cleanUp() first.

Removed an adapter for the async cache loader as not needed, dropping
an instance field.

Simplified some serialization code by removing redundancies.

Run Details

6612 of 7022 relevant lines covered (94.16%)

0.94 hits per line

ben-manes / caffeine / #2838 / 1
100%
master: 100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job #2838.1

ben-manes / caffeine / #2838 / 1 100% master: 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job #2838.1

ben-manes / caffeine / #2838 / 1
100%
master: 100%

README BADGES
x