Why use atomic.AddInt32() and sync.Mutex at the same time about pond HOT 2 CLOSED

Hey @zhu121,

The reason the mutex lock is needed there is to prevent decrementWorkerCount decreasing the workerCount variable while this block is executed in incrementWorkerCount, which could bring undesirable consequences, such as adding more workers than necessary or not starting new ones when needed. And for this to happen, the runningWorkerCount variable in that block must reflect the actual number of worker goroutines during the entirety of the block. A similar thing could happen if incrementWorkerCount was executed simultaneously from 2 different threads.
Does it make sense?

from pond.

Hey @zhu121,

The reason the mutex lock is needed there is to prevent decrementWorkerCount decreasing the workerCount variable while this block is executed in incrementWorkerCount, which could bring undesirable consequences, such as adding more workers than necessary or not starting new ones when needed. And for this to happen, the runningWorkerCount variable in that block must reflect the actual number of worker goroutines during the entirety of the block. A similar thing could happen if incrementWorkerCount was executed simultaneously from 2 different threads. Does it make sense?

Hey @alitto, thank you for your reply, I understand it

from pond.