Incorrectly count redis active connection

[debbyku]

Missing the error checking when incrementing the redis active connection count.

As shown in following codes, the "PoolConnCreated" can be error. I hit such bug when I use password to connect to redis cluster but the whole cluster stops.

(https://github.com/mediocregopher/radix/blob/b6f646dba0b862f3c335167c92d3256a2989406d/trace/pool.go#L61)

type PoolConnCreated struct {
	PoolCommon

	// The reason the connection was created.
	Reason PoolConnCreatedReason

	// How long it took to create the connection.
	ConnectTime time.Duration

	// If connection creation failed, this is the error it failed with.
	Err error
} 

[ysawa0]

lgtm!

[debbyku]

thanks 在 2021年11月27日週六 05:32，Yuki Sawa ***@***.***> 寫道：

…

***@***.**** approved this pull request. lgtm! — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#312 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AHWBE5TN22IZPF4E3TOFJFTUN74HBANCNFSM5IMDMNHQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.