dgraph-io / ristretto / 6035583672

74%

push

set missing Expiration field on evicted items (#345)

On our use case we need to rely on expiration timestamps but we realized
that they return zero timestamps. Our assumption is that this is due to
a bug and not by design.

## Problem
When `OnEvict` callback is executed `Item` does not have `Expiration`
field set. Considering it is a public field this needs to be set and
available.

## Solution
Initialize `Expiration` field before calling the `onEvict` callback

6 of 6 new or added lines in 1 file covered. (100.0%)

2223 of 3016 relevant lines covered (73.71%)

2506912.75 hits per line