Update reconstruction status on PendingComponents from cache instead of the clone.

Author: jimmygchen

beacon_node/beacon_chain/src/data_availability_checker/overflow_lru_cache.rs

@@ -868,7 +868,7 @@ impl<T: BeaconChainTypes> OverflowLRUCache<T> {
     ) -> Result<Option<AvailabilityAndReconstructedColumns<T::EthSpec>>, AvailabilityCheckError>
     {
         // Clone the pending components, so we don't hold the read lock during reconstruction
-        let Some(mut pending_components) = self
+        let Some(pending_components) = self
             .peek_pending_components(&block_root, |pending_components_opt| {
                 pending_components_opt.cloned()
             })
@@ -882,7 +882,14 @@ impl<T: BeaconChainTypes> OverflowLRUCache<T> {
             .map(|r| self.should_reconstruct(&r, &pending_components))?;
 
         if should_reconstruct {
-            pending_components.reconstruction_started();
+            self.critical
+                .write()
+                .in_memory
+                .get_mut(&block_root)
+                .map(|pending_components| {
+                    pending_components.reconstruction_started();
+                });
+
             metrics::inc_counter(&KZG_DATA_COLUMN_RECONSTRUCTION_ATTEMPTS);
             let timer = metrics::start_timer(&metrics::DATA_AVAILABILITY_RECONSTRUCTION_TIME);