review suggestions

celestiaorg · Apr 19, 2023 · 8ae29dc · 8ae29dc
1 parent 02f9eb1
commit 8ae29dc
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/share/p2p/peers/pool.go b/share/p2p/peers/pool.go
@@ -12,7 +12,7 @@ const defaultCleanupThreshold = 2
 
 // pool stores peers and provides methods for simple round-robin access.
 type pool struct {
-	m           sync.Mutex
+	m           sync.RWMutex
 	peersList   []peer.ID
 	statuses    map[peer.ID]status
 	cooldown    *timedQueue
@@ -196,7 +196,7 @@ func (p *pool) checkHasPeers() {
 }
 
 func (p *pool) size() int {
-	p.m.Lock()
-	defer p.m.Unlock()
+	p.m.RLock()
+	defer p.m.RUnlock()
 	return p.activeCount
 }
diff --git a/share/p2p/shrexnd/client.go b/share/p2p/shrexnd/client.go
@@ -69,7 +69,7 @@ func (c *Client) RequestND(
 		}
 	}
 	if err != p2p.ErrNotFound {
-		log.Warnw("client: peer returned err", "err", err)
+		log.Warnw("client-nd: peer returned err", "err", err)
 	}
 	return nil, err
 }