Add comment and logging.

beacon_node/beacon_chain/src/data_availability_checker/overflow_lru_cache.rs

@@ -390,6 +390,9 @@ pub struct DataAvailabilityCheckerInner<T: BeaconChainTypes> {
     spec: Arc<ChainSpec>,
 }
 
+// This enum is only used internally within the crate in the reconstruction function to improve
+// readability, so it's OK to not box the variant value, and it shouldn't impact memory much with
+// the current usage, as it's deconstructed immediately.
 #[allow(clippy::large_enum_variant)]
 pub(crate) enum ReconstructColumnsDecision<E: EthSpec> {
     Yes(PendingComponents<E>),

beacon_node/network/src/network_beacon_processor/mod.rs

@@ -911,6 +911,7 @@ impl<T: BeaconChainTypes> NetworkBeaconProcessor<T> {
                 error!(
                     self.log,
                     "Error during data column reconstruction";
+                    "block_root" => %block_root,
                     "error" => ?e
                 );
                 None