luxolus/open-nomad
2
0
Fork 0
Code Issues 1 Pull requests Packages Releases Wiki Activity

fix: use NewSafeTimer

Browse source 
Signed-off-by: Yoan Blanc <yoan@dosimple.ch>
This commit is contained in:
Yoan Blanc 2022-04-11 19:37:14 +02:00
parent 5e8254beda
commit 3e79d58e4a
No known key found for this signature in database
GPG key ID: 6058CF4574298812
1 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
client/rpc.go
View file

@ -109,8 +109,11 @@ TRY:
} }
// Wait to avoid thundering herd // Wait to avoid thundering herd
timer, cancel := helper.NewSafeTimer(helper.RandomStagger(c.config.RPCHoldTimeout / structs.JitterFraction))
defer cancel()
select { select {
case <-time.After(helper.RandomStagger(c.config.RPCHoldTimeout / structs.JitterFraction)): case <-timer.C:
// If we are going to retry a blocking query we need to update the time to block so it finishes by our deadline. // If we are going to retry a blocking query we need to update the time to block so it finishes by our deadline.
if info, ok := args.(structs.RPCInfo); ok && info.TimeToBlock() > 0 { if info, ok := args.(structs.RPCInfo); ok && info.TimeToBlock() > 0 {
newBlockTime := time.Until(deadline) newBlockTime := time.Until(deadline)