From 22831326589815bd4ba1ee205da964fd407cab25 Mon Sep 17 00:00:00 2001 From: axelKingsley Date: Tue, 6 May 2025 17:29:02 -0500 Subject: [PATCH] Panic on Failed to Enqueue --- op-node/rollup/event/system.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/op-node/rollup/event/system.go b/op-node/rollup/event/system.go index d0ae0e3d02d42..be942cbd5c6bd 100644 --- a/op-node/rollup/event/system.go +++ b/op-node/rollup/event/system.go @@ -262,8 +262,11 @@ func (s *Sys) emit(name string, derivContext uint64, ev Event) { s.recordEmit(name, annotated, derivContext, emitTime) err := s.executor.Enqueue(annotated) + // If the event system cannot enqueue an event, then it is a critical error + // and we should panic to avoid deferred errors creating behaviors that are hard to reason about. + // The Sys cannot decide if an event is important or not, so all events should be considered critical. if err != nil { s.log.Error("Failed to enqueue event", "emitter", name, "event", ev, "context", derivContext) - return + panic(err) } }