Fix obvious mistakes

Author: dknopik

beacon_node/beacon_chain/src/data_availability_checker.rs

@@ -450,9 +450,8 @@ impl<T: BeaconChainTypes> DataAvailabilityChecker<T> {
         executed_block: AvailabilityPendingExecutedBlock<T::EthSpec>,
     ) -> Result<Availability<T::EthSpec>, AvailabilityCheckError> {
         let block = executed_block.as_block();
-        self.partial_assembler
-            .init(block.canonical_root(), block)
-            .map_err(Error::PartialAssemblerError)?;
+        // Ignore error which happens at pre-fulu blocks
+        let _ = self.partial_assembler.init(block.canonical_root(), block);
         self.availability_cache.put_executed_block(executed_block)
     }
 
@@ -464,9 +463,8 @@ impl<T: BeaconChainTypes> DataAvailabilityChecker<T> {
         block: Arc<SignedBeaconBlock<T::EthSpec>>,
         source: BlockImportSource,
     ) -> Result<(), Error> {
-        self.partial_assembler
-            .init(block_root, block.as_ref())
-            .map_err(Error::PartialAssemblerError)?;
+        // Ignore error which happens at pre-fulu blocks
+        let _ = self.partial_assembler.init(block_root, block.as_ref());
         self.availability_cache
             .put_pre_execution_block(block_root, block, source)
     }

beacon_node/beacon_chain/src/partial_data_column_assembler.rs

@@ -40,7 +40,7 @@ impl<E: EthSpec> PartialDataColumnAssembler<E> {
     where
         T: TryInto<PartialDataColumnHeader<E>>,
     {
-        let assemblies = self.assemblies.write();
+        let mut assemblies = self.assemblies.write();
 
         if assemblies.contains(&block_root) {
             return Ok(false);
@@ -53,7 +53,7 @@ impl<E: EthSpec> PartialDataColumnAssembler<E> {
             columns: HashMap::new(),
         };
 
-        self.assemblies.write().put(block_root, assembly);
+        assemblies.put(block_root, assembly);
 
         Ok(true)
     }

crypto/kzg/src/lib.rs

@@ -13,17 +13,17 @@ pub use crate::{
 };
 
 pub use c_kzg::{
-    BYTES_PER_BLOB, BYTES_PER_COMMITMENT, BYTES_PER_FIELD_ELEMENT, BYTES_PER_PROOF, Blob, Bytes32,
-    Bytes48, FIELD_ELEMENTS_PER_BLOB, KzgSettings,
+    Blob, Bytes32, Bytes48, KzgSettings, BYTES_PER_BLOB, BYTES_PER_COMMITMENT,
+    BYTES_PER_FIELD_ELEMENT, BYTES_PER_PROOF, FIELD_ELEMENTS_PER_BLOB,
 };
 
 use crate::trusted_setup::load_trusted_setup;
 use rayon::prelude::*;
 pub use rust_eth_kzg::{
-    Cell, CellIndex as CellID, CellRef, TrustedSetup as PeerDASTrustedSetup,
     constants::{BYTES_PER_CELL, CELLS_PER_EXT_BLOB},
+    Cell, CellIndex as CellID, CellRef, TrustedSetup as PeerDASTrustedSetup,
 };
-use tracing::{Span, instrument};
+use tracing::{instrument, Span};
 
 /// Disables the fixed-base multi-scalar multiplication optimization for computing
 /// cell KZG proofs, because `rust-eth-kzg` already handles the precomputation.