recursor: when replacing an expired entry, move it to the back #3352

Merged
merged 1 commit into from Feb 20, 2016

Projects

None yet

3 participants

@rgacogne
Member

If we did a lookup earlier and found the entry to be expired,
we moved it to the front of the expunge queue. If we don't move
it to the back, and the next cache purging operation occurs before
a cache hit using this entry, it's very likely to get expunged.

@Habbie
Member
Habbie commented Feb 10, 2016

I merged #3354, can you rebase?

@rgacogne rgacogne recursor: when replacing an expired entry, move it to the back
If we did a lookup earlier and found the entry to be expired,
we moved it to the front of the expunge queue. If we don't move
it to the back, and the next cache purging operation occurs before
a cache hit using this entry, it's very likely to get expunged.
2b94bb4
@rgacogne
Member

rebased and travis passed!

@Habbie
Member
Habbie commented Feb 10, 2016

Currently being tested by user.

@ahupowerdns ahupowerdns merged commit d5ae7bc into PowerDNS:rel/rec-3.7.4 Feb 20, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@rgacogne rgacogne deleted the rgacogne:rec-cache-eviction branch Feb 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment