Skip to content

Commit

Permalink
Add comment about PopulateFeedBacklog in seq coordinator being redundant
Browse files Browse the repository at this point in the history
  • Loading branch information
PlasmaPower committed Dec 21, 2023
1 parent 4e73be7 commit ae54e08
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions arbnode/seq_coordinator.go
Original file line number Diff line number Diff line change
Expand Up @@ -650,6 +650,8 @@ func (c *SeqCoordinator) update(ctx context.Context) time.Duration {
log.Warn("failed sequencing delayed messages after catching lock", "err", err)
}
}
// This should be redundant now that even non-primary sequencers broadcast over the feed,
// but the backlog efficiently deduplicates messages, so better safe than sorry.
err = c.streamer.PopulateFeedBacklog()
if err != nil {
log.Warn("failed to populate the feed backlog on lockout acquisition", "err", err)
Expand Down

0 comments on commit ae54e08

Please sign in to comment.