Skip to content

Commit 9511446

Browse files
committed
rebase fixes
1 parent f283833 commit 9511446

1 file changed

Lines changed: 6 additions & 9 deletions

File tree

yarn-project/sequencer-client/src/sequencer/sequencer.ts

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -373,19 +373,19 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
373373
const pendingCheckpointNumber = await this.l2BlockSource.getPendingCheckpointNumber();
374374
if (
375375
this.epochCache.isProposerPipeliningEnabled() &&
376-
pendingCheckpointNumber === syncedTo.checkpointedCheckpointNumber + 1
376+
pendingCheckpointNumber === syncedTo.latestCheckpointNumber + 1
377377
) {
378378
this.log.verbose(`Building on top of pending checkpoint ${pendingCheckpointNumber}`);
379379
} else if (
380380
this.epochCache.isProposerPipeliningEnabled() &&
381-
syncedTo.checkpointNumber > syncedTo.checkpointedCheckpointNumber
381+
syncedTo.checkpointNumber > syncedTo.latestCheckpointNumber
382382
) {
383383
// Pipelining is enabled and there are uncheckpointed blocks beyond the confirmed chain,
384384
// but the pending checkpoint attestation hasn't arrived yet (skip condition above failed).
385385
// Wait for the pending checkpoint to be attested or for uncheckpointed blocks to be pruned
386386
// rather than making an L1 call that would fail or give misleading results.
387387
this.log.warn(
388-
`Pipelining stall: proposed chain at checkpoint ${syncedTo.checkpointNumber} is ahead of confirmed ${syncedTo.checkpointedCheckpointNumber} but no pending attestation (pending=${pendingCheckpointNumber}). Waiting for reconciliation.`,
388+
`Pipelining stall: proposed chain at checkpoint ${syncedTo.checkpointNumber} is ahead of confirmed ${syncedTo.latestCheckpointNumber} but no pending attestation (pending=${pendingCheckpointNumber}). Waiting for reconciliation.`,
389389
logCtx,
390390
);
391391
this.metrics.recordCheckpointPrecheckFailed('pipelining_stall');
@@ -619,9 +619,8 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
619619
archive,
620620
syncedL2Slot,
621621
checkpointedArchive: archive,
622-
l1Timestamp,
622+
latestCheckpointNumber: l2Tips.checkpointed.checkpoint.number,
623623
pendingChainValidationStatus,
624-
checkpointedCheckpointNumber: l2Tips.checkpointed.checkpoint.number,
625624
};
626625
}
627626

@@ -652,9 +651,8 @@ export class Sequencer extends (EventEmitter as new () => TypedEventEmitter<Sequ
652651
archive: blockData.archive.root,
653652
syncedL2Slot,
654653
checkpointedArchive,
655-
l1Timestamp,
654+
latestCheckpointNumber: l2Tips.checkpointed.checkpoint.number,
656655
pendingChainValidationStatus,
657-
checkpointedCheckpointNumber: l2Tips.checkpointed.checkpoint.number,
658656
};
659657
}
660658

@@ -1016,7 +1014,6 @@ type SequencerSyncCheckResult = {
10161014
syncedL2Slot: SlotNumber;
10171015
/** Archive root of the last L1-confirmed (checkpointed) block. Used for L1 canProposeAt checks. */
10181016
checkpointedArchive: Fr;
1019-
l1Timestamp: bigint;
1017+
latestCheckpointNumber: CheckpointNumber;
10201018
pendingChainValidationStatus: ValidateCheckpointResult;
1021-
checkpointedCheckpointNumber: CheckpointNumber;
10221019
};

0 commit comments

Comments
 (0)