Skip to content
This repository was archived by the owner on Mar 8, 2025. It is now read-only.

Commit 5867c1d

Browse files
committed
fix: emit CommitPayloadEvent before GetMinimizedPayload to avoid not being cleared in time
1 parent b3a23e3 commit 5867c1d

File tree

1 file changed

+6
-7
lines changed

1 file changed

+6
-7
lines changed

op-node/rollup/engine/build_seal.go

+6-7
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,12 @@ func (eq *EngDeriver) onBuildSeal(ev BuildSealEvent) {
5757
defer cancel()
5858

5959
sealingStart := time.Now()
60+
eq.emitter.Emit(conductor.CommitPayloadEvent{
61+
IsLastInSpan: ev.IsLastInSpan,
62+
DerivedFrom: ev.DerivedFrom,
63+
Info: ev.Info,
64+
Ref: eth.L2BlockRef{},
65+
})
6066
envelope, err := eq.ec.engine.GetMinimizedPayload(ctx, ev.Info)
6167
if err != nil {
6268
if x, ok := err.(eth.InputError); ok && x.Code == eth.UnknownPayload { //nolint:all
@@ -101,13 +107,6 @@ func (eq *EngDeriver) onBuildSeal(ev BuildSealEvent) {
101107
return
102108
}
103109

104-
eq.emitter.Emit(conductor.CommitPayloadEvent{
105-
IsLastInSpan: ev.IsLastInSpan,
106-
DerivedFrom: ev.DerivedFrom,
107-
Info: ev.Info,
108-
Ref: ref,
109-
})
110-
111110
now := time.Now()
112111
sealTime := now.Sub(sealingStart)
113112
buildTime := now.Sub(ev.BuildStarted)

0 commit comments

Comments
 (0)