aarifullin/frostfs-node
1
0
Fork 0
forked from TrueCloudLab/frostfs-node
Code Issues Pull requests Projects Releases Packages Wiki Activity

[#274] wc: Make wait groups work more explicit

Browse source 
Do not run routine that calls `wg.Done()` inside, it is hard to read.

Signed-off-by: Pavel Karpy <p.karpy@yadro.com>
This commit is contained in:
Pavel Karpy 2023-04-20 17:59:44 +03:00
parent 4f5f832137
commit 6b6f33ed71
1 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
pkg/local_object_storage/writecache/flush.go
View file

@ -41,7 +41,10 @@ func (c *cache) runFlushLoop() {
} }
c.wg.Add(1) c.wg.Add(1)
go c.flushBigObjects(context.TODO()) go func() {
c.flushBigObjects(context.TODO())
c.wg.Done()
}()
c.wg.Add(1) c.wg.Add(1)
go func() { go func() {
@ -146,8 +149,6 @@ func (c *cache) flushDB() {
} }
func (c *cache) flushBigObjects(ctx context.Context) { func (c *cache) flushBigObjects(ctx context.Context) {
defer c.wg.Done()
tick := time.NewTicker(defaultFlushInterval * 10) tick := time.NewTicker(defaultFlushInterval * 10)
for { for {
select { select {