EliChin/frostfs-s3-gw
1
0
Fork 0
forked from TrueCloudLab/frostfs-s3-gw
Code Issues Pull requests Projects Releases Packages Wiki Activity

[#206] Refactor ListObjectsCache

Browse source 
Replaced conflicting names

Signed-off-by: Angira Kekteeva <kira@nspcc.ru>
This commit is contained in:
Angira Kekteeva 2021-08-10 15:23:55 +03:00 committed by Kirillov Denis
parent 4ca56d3d6e
commit ae19eb4ad4
4 changed files with 15 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
api/layer/layer.go
View file

@ -25,7 +25,8 @@ type (
layer struct { layer struct {
pool pool.Pool pool pool.Pool
log *zap.Logger log *zap.Logger
cache ObjectsListCache listObjCache ObjectsListCache
} }
// Params stores basic API parameters. // Params stores basic API parameters.
@ -132,7 +133,7 @@ func NewLayer(log *zap.Logger, conns pool.Pool) Client {
return &layer{ return &layer{
pool: conns, pool: conns,
log: log, log: log,
cache: newListObjectsCache(defaultCacheLifetime), listObjCache: newListObjectsCache(defaultObjectsListCacheLifetime),
} }
} }

4
api/layer/object.go
View file

@ -374,7 +374,7 @@ func (n *layer) listAllObjects(ctx context.Context, p ListObjectsParamsCommon) (
return nil, err return nil, err
} }
allObjects = n.cache.Get(cacheKey) allObjects = n.listObjCache.Get(cacheKey)
if allObjects == nil { if allObjects == nil {
allObjects, err = n.listSortedObjectsFromNeoFS(ctx, allObjectParams{ allObjects, err = n.listSortedObjectsFromNeoFS(ctx, allObjectParams{
@ -387,7 +387,7 @@ func (n *layer) listAllObjects(ctx context.Context, p ListObjectsParamsCommon) (
} }
// putting to cache a copy of allObjects because allObjects can be modified further // putting to cache a copy of allObjects because allObjects can be modified further
n.cache.Put(cacheKey, append([]*ObjectInfo(nil), allObjects...)) n.listObjCache.Put(cacheKey, append([]*ObjectInfo(nil), allObjects...))
} }
return allObjects, nil return allObjects, nil

12
api/layer/object_cache.go → api/layer/object_list_cache.go
View file

@ -13,7 +13,7 @@ import (
request. request.
The cache is a map which has a key: cacheOptions struct and a value: list of objects. After putting a record we The cache is a map which has a key: cacheOptions struct and a value: list of objects. After putting a record we
start a timer (via time.AfterFunc) that removes the record after defaultCacheLifetime value. start a timer (via time.AfterFunc) that removes the record after defaultObjectsListCacheLifetime value.
When we get a request from the user we just try to find the suitable and non-expired cache and then we return When we get a request from the user we just try to find the suitable and non-expired cache and then we return
the list of objects. Otherwise we send the request to NeoFS. the list of objects. Otherwise we send the request to NeoFS.
@ -27,15 +27,15 @@ type (
} }
) )
const defaultCacheLifetime = time.Second * 60 const defaultObjectsListCacheLifetime = time.Second * 60
type ( type (
listObjectsCache struct { listObjectsCache struct {
cacheLifetime time.Duration cacheLifetime time.Duration
caches map[cacheOptions]cache caches map[cacheOptions]cacheEntry
mtx sync.RWMutex mtx sync.RWMutex
} }
cache struct { cacheEntry struct {
list []*ObjectInfo list []*ObjectInfo
} }
cacheOptions struct { cacheOptions struct {
@ -47,7 +47,7 @@ type (
func newListObjectsCache(lifetime time.Duration) *listObjectsCache { func newListObjectsCache(lifetime time.Duration) *listObjectsCache {
return &listObjectsCache{ return &listObjectsCache{
caches: make(map[cacheOptions]cache), caches: make(map[cacheOptions]cacheEntry),
cacheLifetime: lifetime, cacheLifetime: lifetime,
} }
} }
@ -65,7 +65,7 @@ func (l *listObjectsCache) Put(key cacheOptions, objects []*ObjectInfo) {
if len(objects) == 0 { if len(objects) == 0 {
return return
} }
var c cache var c cacheEntry
l.mtx.Lock() l.mtx.Lock()
defer l.mtx.Unlock() defer l.mtx.Unlock()
c.list = objects c.list = objects

0
api/layer/object_cache_test.go → api/layer/object_list_cache_test.go
View file