Merge branch 'develop' of https://github.com/prysmaticlabs/geth-sharding into exitReconstructionEarly

beacon-chain/sync/subscriber_beacon_blocks.go

@@ -87,11 +87,24 @@ func (s *Service) reconstructAndBroadcastBlobs(ctx context.Context, block interf
 		log.WithError(err).Error("Failed to read commitments from block")
 		return
 	}
+	// Exit early if there are no commitments in block.
+	if len(cmts) == 0 {
+		return
+	}
+
+	var blobUnseen bool
 	for i := range cmts {
 		if summary.HasIndex(uint64(i)) {
 			blobExistedInDBTotal.Inc()
+		} else {
+			blobUnseen = true
 		}
 	}
+	// If all blobs have been seen via gossip, we can exit the reconstruction
+	// routine early as there is nothing to fetch from the EL.
+	if !blobUnseen {
+		return
+	}
 
 	// Reconstruct blob sidecars from the EL
 	blobSidecars, err := s.cfg.executionReconstructor.ReconstructBlobSidecars(ctx, block, blockRoot, summary.HasIndex)

changelog/nisdas_exit_reconstruction_early.md

@@ -0,0 +1,3 @@
+### Changed
+
+- Exit Blob Reconstruction early in the event we have already seen them.