fix: transfer cache in prepareNextSlot epoch transition (#6068)

ChainSafe · Nov 8, 2023 · c8e6b67 · c8e6b67
1 parent d3f40d2
commit c8e6b67
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 8 deletions.
diff --git a/packages/beacon-node/src/chain/prepareNextSlot.ts b/packages/beacon-node/src/chain/prepareNextSlot.ts
@@ -98,7 +98,8 @@ export class PrepareNextSlotScheduler {
       const prepareState = await this.chain.regen.getBlockSlotState(
         headRoot,
         prepareSlot,
-        {dontTransferCache: true},
+        // the 1st slot of next epoch will likely use this Previous Root Checkpoint state so we transfer cache here
+        {dontTransferCache: false},
         RegenCaller.precomputeEpoch
       );
 

diff --git a/packages/beacon-node/src/chain/regen/queued.ts b/packages/beacon-node/src/chain/regen/queued.ts
@@ -149,14 +149,7 @@ export class QueuedStateRegenerator implements IStateRegenerator {
     if (parentEpoch < blockEpoch) {
       const checkpointState = this.checkpointStateCache.getLatest(parentRoot, blockEpoch);
       if (checkpointState && computeEpochAtSlot(checkpointState.slot) === blockEpoch) {
-        // TODO: Miss-use of checkpointStateCache here
         return checkpointState;
-        // console.error({
-        //   "checkpointState.slot": checkpointState.slot,
-        //   "block.slot": block.slot,
-        //   blockEpoch,
-        //   blockEpochStartSlot: computeStartSlotAtEpoch(blockEpoch),
-        // });
       }
     }