Skip to content

eth: not begin to sync during processing broadcasted blocks or hash #3167

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
buddh0 merged 2 commits into from
Jun 16, 2025
Merged

eth: not begin to sync during processing broadcasted blocks or hash #3167

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
40927fb
eth: not begin to sync during processing broadcasted blocks or hash
buddh0 Jun 13, 2025
5a43750
eth: downgrade log level for catching up
buddh0 Jun 14, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 13 additions & 0 deletions eth/sync.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,20 @@ func (cs *chainSyncer) nextSyncOp() *chainSyncOp {
// cs.handler.acceptTxs.Store(false)
// log.Info("Disable transaction acceptance randomly for the delay exceeding 10 blocks.")
// }
} else if op.td.Cmp(new(big.Int).Add(ourTD, common.Big2)) <= 0 { // common.Big2: difficulty of an in-turn block
// On BSC, blocks are produced much faster than on Ethereum.
// If the node is only slightly behind (e.g., 1 block), syncing is unnecessary.
// It's likely still processing broadcasted blocks or block hash announcements.
// In most cases, the node will catch up within 3 seconds.
time.Sleep(3 * time.Second)

// Re-check local head to see if it has caught up
if _, latestTD := cs.modeAndLocalHead(); ourTD.Cmp(latestTD) < 0 {
log.Trace("The local head is already caught up; synchronization is not required.")
return nil
}
}

return op
}

Expand Down