TrueCloudLab/frostfs-node
20
1
Fork 27
Code Issues 100 Pull requests 4 Projects Releases 1 Activity Actions

[#2158] policer: Reduce default cache size

Browse source 
We use cache to avoid policing the same object multiple times in a short
time span (< 30 seconds). If we have 200_000 objects in a blobstor, it is a bit useless
-- if it takes 1 second to process an object and we have `replicator.pool_size: 20`
in config, the next iteration will happen in 10_000 second which is much
larger than 30 second. However we still consume a lot of memory, so it
makes sense to use saner default.

Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>
This commit is contained in:
Evgenii Stratonikov 2022-12-14 18:20:11 +03:00 committed by Anton Nikiforov
parent 0244f2e5ce
commit b207dc424f
2 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
CHANGELOG.md
View file

@ -20,7 +20,8 @@ Changelog for NeoFS Node
- `neofs-cli container get-eacl` print ACL table in json format only with arg `--json' (#2012) - `neofs-cli container get-eacl` print ACL table in json format only with arg `--json' (#2012)
- Side chain notary deposits use max uint32 as till parameter (#1486) - Side chain notary deposits use max uint32 as till parameter (#1486)
- Allow object removal without linking object (#2100) - Allow object removal without linking object (#2100)
- `neofs-cli container delete` command pre-checks container ownership (#2106) - `neofs-cli container delete` command pre-checks container ownership (#2106)
- Policer cache size is now 1024 (#2158)
### Fixed ### Fixed
- Open FSTree in sync mode by default (#1992) - Open FSTree in sync mode by default (#1992)

2
pkg/services/policer/policer.go
View file

@ -99,7 +99,7 @@ func defaultCfg() *cfg {
return &cfg{ return &cfg{
log: &logger.Logger{Logger: zap.L()}, log: &logger.Logger{Logger: zap.L()},
batchSize: 10, batchSize: 10,
cacheSize: 200_000, // should not allocate more than 200 MiB cacheSize: 1024, // 1024 * address size = 1024 * 64 = 64 MiB
rebalanceFreq: 1 * time.Second, rebalanceFreq: 1 * time.Second,
evictDuration: 30 * time.Second, evictDuration: 30 * time.Second,
} }