dstepanov-yadro/frostfs-sdk-go
1
0
Fork 0
forked from TrueCloudLab/frostfs-sdk-go
Code Pull requests Activity

[#300] pool/cm: Remove unused mutex in 'statistics'

Browse source 
Signed-off-by: Alexander Chuprov <a.chuprov@yadro.com>
This commit is contained in:
Alexander Chuprov 2025-03-05 15:30:48 +03:00 • committed by Evgenii Stratonikov
parent 56046935b0
commit 17177697b5
1 changed files with 0 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
pool/connection_manager.go
View file

@ -297,7 +297,6 @@ func (cm connectionManager) Statistic() Statistic {
stat := Statistic{} stat := Statistic{}
for _, inner := range cm.innerPools { for _, inner := range cm.innerPools {
nodes := make([]string, 0, len(inner.clients)) nodes := make([]string, 0, len(inner.clients))
inner.lock.RLock()
for _, cl := range inner.clients { for _, cl := range inner.clients {
if cl.isHealthy() { if cl.isHealthy() {
nodes = append(nodes, cl.address()) nodes = append(nodes, cl.address())
@ -311,7 +310,6 @@ func (cm connectionManager) Statistic() Statistic {
stat.nodes = append(stat.nodes, node) stat.nodes = append(stat.nodes, node)
stat.overallErrors += node.overallErrors stat.overallErrors += node.overallErrors
} }
inner.lock.RUnlock()
if len(stat.currentNodes) == 0 { if len(stat.currentNodes) == 0 {
stat.currentNodes = nodes stat.currentNodes = nodes
} }