add comment

status-im · Dec 3, 2024 · 6f0880f · 6f0880f
1 parent b9befdb
commit 6f0880f
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/beacon_chain/sync/request_manager.nim b/beacon_chain/sync/request_manager.nim
@@ -274,10 +274,14 @@ proc fetchBlobsFromNetwork(self: RequestManager,
 proc checkPeerCustody*(rman: RequestManager,
                        peer: Peer):
                        bool =
-  ## Returns true if the peer custodies atleast
-  ## ONE of the common custody columns, straight 
-  ## away returns true if the peer is a supernode.
+  # Returns true if the peer custodies atleast
+  # ONE of the common custody columns, straight 
+  # away returns true if the peer is a supernode.
   if rman.supernode:
+    # For a supernode, it is always best/optimistic
+    # to filter other supernodes, rather than filter
+    # too many full nodes that have a subset of the custody
+    # columns
     if peer.lookupCscFromPeer() == 
         DATA_COLUMN_SIDECAR_SUBNET_COUNT.uint64:
       return true