diff options
author | 2017-06-13 12:39:10 -0700 | |
---|---|---|
committer | 2017-06-13 12:39:10 -0700 | |
commit | e9eda7e7c8ed75d62b02f23c62e8e318ea1685ae (patch) | |
tree | 152a04673698e1a39caa751c02612d4e21315662 /middleware/dnssec/dnssec.go | |
parent | b1efd3736e6e68ab01baf54f83071c62690899b2 (diff) | |
download | coredns-e9eda7e7c8ed75d62b02f23c62e8e318ea1685ae.tar.gz coredns-e9eda7e7c8ed75d62b02f23c62e8e318ea1685ae.tar.zst coredns-e9eda7e7c8ed75d62b02f23c62e8e318ea1685ae.zip |
New cache implementation and prefetch handing in mw/cache (#731)
* cache: add sharded cache implementation
Add Cache impl and a few tests. This cache is 256-way sharded, mainly
so each shard has it's own lock. The main cache structure is a readonly
jump plane into the right shard.
This should remove the single lock contention on the main lock and
provide more concurrent throughput - Obviously this hasn't been tested
or measured.
The key into the cache was made a uint32 (hash.fnv) and the hashing op
is not using strings.ToLower anymore remove any GC in that code path.
* here too
* Minimum shard size
* typos
* blurp
* small cleanups no defer
* typo
* Add freq based on Johns idea
* cherry-pick conflict resolv
* typo
* update from early code review from john
* add prefetch to the cache
* mw/cache: add prefetch
* remove println
* remove comment
* Fix tests
* Test prefetch in setup
* Add start of cache
* try add diff cache options
* Add hacky testcase
* not needed
* allow the use of a percentage for prefetch
If the TTL falls below xx% do a prefetch, if the record was popular.
Some other fixes and correctly prefetch only popular records.
Diffstat (limited to 'middleware/dnssec/dnssec.go')
-rw-r--r-- | middleware/dnssec/dnssec.go | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/middleware/dnssec/dnssec.go b/middleware/dnssec/dnssec.go index 7ad5f2bf7..4e1e70217 100644 --- a/middleware/dnssec/dnssec.go +++ b/middleware/dnssec/dnssec.go @@ -6,11 +6,11 @@ import ( "time" "github.com/coredns/coredns/middleware" + "github.com/coredns/coredns/middleware/pkg/cache" "github.com/coredns/coredns/middleware/pkg/response" "github.com/coredns/coredns/middleware/pkg/singleflight" "github.com/coredns/coredns/request" - "github.com/hashicorp/golang-lru" "github.com/miekg/dns" ) @@ -21,15 +21,15 @@ type Dnssec struct { zones []string keys []*DNSKEY inflight *singleflight.Group - cache *lru.Cache + cache *cache.Cache } // New returns a new Dnssec. -func New(zones []string, keys []*DNSKEY, next middleware.Handler, cache *lru.Cache) Dnssec { +func New(zones []string, keys []*DNSKEY, next middleware.Handler, c *cache.Cache) Dnssec { return Dnssec{Next: next, zones: zones, keys: keys, - cache: cache, + cache: c, inflight: new(singleflight.Group), } } @@ -90,7 +90,7 @@ func (d Dnssec) Sign(state request.Request, zone string, now time.Time) *dns.Msg } func (d Dnssec) sign(rrs []dns.RR, signerName string, ttl, incep, expir uint32) ([]dns.RR, error) { - k := key(rrs) + k := hash(rrs) sgs, ok := d.get(k) if ok { return sgs, nil @@ -110,11 +110,11 @@ func (d Dnssec) sign(rrs []dns.RR, signerName string, ttl, incep, expir uint32) return sigs.([]dns.RR), err } -func (d Dnssec) set(key string, sigs []dns.RR) { +func (d Dnssec) set(key uint32, sigs []dns.RR) { d.cache.Add(key, sigs) } -func (d Dnssec) get(key string) ([]dns.RR, bool) { +func (d Dnssec) get(key uint32) ([]dns.RR, bool) { if s, ok := d.cache.Get(key); ok { cacheHits.Inc() return s.([]dns.RR), true |