Reduce using mutex when update application settings. #329
Labels
No Label
P0
P1
P2
P3
good first issue
Infrastructure
blocked
bug
config
discussion
documentation
duplicate
enhancement
go
help wanted
internal
invalid
kludge
observability
perfomance
question
refactoring
wontfix
No Milestone
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: TrueCloudLab/frostfs-s3-gw#329
Loading…
Reference in New Issue
There is no content yet.
Delete Branch "%!s(<nil>)"
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?
Is your feature request related to a problem? Please describe.
We have appSettings.update method that fetch and update config params. Each function inside
appSettings.update
acquire lock that in general ok, but we can reduce amount of lock/unlock operations during sighup reload.Describe the solution you'd like
Use just one lock/unlock operation during sighup. So we can change the
appSettings.update
method to look likeappSettings.updateNamespacesSettings
:Describe alternatives you've considered
Don't change anything
Additional context
No