Race condition on minPoolSize and maxIdleTimeMS

[bmarguer]

Hello.
It seems to have a race condition (latest tag : efe0945) on "DialWithInfo" when :

• write part : cluster.minPoolSize is written
• read part : just before "newCluster" has created the goroutine which launch the synServersLoop (this routine calls syncServersIteration --> spawnSync --> cluster.server --> minPoolSize := cluster.minPoolSize)
  I think the minPoolSize and the maxIdleTimeMS should be params of newCluster function like others info attributes ?

[domodwyer]

Hi @bmarguer

This code has been extensively reworked - would you mind trying the development branch and confirming it is still an issue?

Thanks!

Dom

[bmarguer]

Oh sorry ! You're right !
cluster.go is corrected in development branch. There's no more useless read/write in server fucntion and all attributes are encapsulated in dialInfo struct.
Thanks !
I close this issue.

[domodwyer]

Hey @bmarguer

That's great, thanks for opening the issue to check though!

Dom