ape: Add morph chain cache #1044
No reviewers
Labels
No Label
P0
P1
P2
P3
badger
frostfs-adm
frostfs-cli
frostfs-ir
frostfs-lens
frostfs-node
good first issue
triage
Infrastructure
blocked
bug
config
discussion
documentation
duplicate
enhancement
go
help wanted
internal
invalid
kludge
observability
perfomance
question
refactoring
wontfix
No Milestone
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: TrueCloudLab/frostfs-node#1044
Loading…
Reference in New Issue
There is no content yet.
Delete Branch "dstepanov-yadro/frostfs-node:fix/get_performance"
Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?
Morph chain cache added. It is required to reduce
get
latency.9b5c58f9df
to93af379bab
93af379bab
to500f327fe9
500f327fe9
to73afc06913
73afc06913
to18170a3153
@ -0,0 +28,4 @@
// from "access_policy_engine" section.
//
// Returns MorphCacheTTLDefault if the value is not positive duration.
func MorphCacheTTL(c *config.Config) time.Duration {
We already have
c.cfgMorph.cacheTTL = morphconfig.CacheTTL(c.appCfg)
, why wasn't it used?It also has 0 default and is adjusted to be taken as TimePerBlock, 5 second is a magic constant in comparison.
right, fixed.
18170a3153
toa54fc52281
@ -0,0 +15,4 @@
// from "access_policy_engine" section.
//
// Returns false if the value is not valid bool.
func MorphCacheEnabled(c *config.Config) bool {
Other TTL caches are always enabled, maybe stick with this policy here too?
It clearly has benefits (and again, enabling it explicitly now is another task on our platform team).
Done
Other then the comment LGTM
a54fc52281
to0146c033a0
0146c033a0
to35dbe84124
35dbe84124
to4690c4ac4c
4690c4ac4c
to5d58b44bc8