@cryptotaxi247 / kubo / commits / 66a031f17

blockstore: move ARC cache below the bloom cache

ARC cache is influenced by requests and bloom isn't This means that if bloom is able to remove some requests caching them in ARC is pointless. License: MIT Signed-off-by: Jakub Sztandera <kubuxu@protonmail.ch>

Jakub Sztandera committed Sep 5, 2016 at 17:05 UTC 66a031f17cb26d094648d0001120926eee01b6ca
1 file changed +3 -3
blocks/blockstore/caching.go
+3 -3
@@ -37,12 +37,12 @@ func CachedBlockstore(bs GCBlockstore,
37
38 ctx = metrics.CtxSubScope(ctx, "bs.cache")
39
40 - if opts.HasBloomFilterSize != 0 {
41 - cbs, err = bloomCached(cbs, ctx, opts.HasBloomFilterSize, opts.HasBloomFilterHashes)
42 - }
40 if opts.HasARCCacheSize > 0 {
41 cbs, err = newARCCachedBS(cbs, ctx, opts.HasARCCacheSize)
42 }
43 + if opts.HasBloomFilterSize != 0 {
44 + cbs, err = bloomCached(cbs, ctx, opts.HasBloomFilterSize, opts.HasBloomFilterHashes)
45 + }
46
47 return cbs, err
48 }