Skip to content

Comments

chore(consensus): Finalize step improvements and fixes#1488

Closed
ancazamfir wants to merge 2 commits intocirclefin:mainfrom
ancazamfir:finalize-fixes
Closed

chore(consensus): Finalize step improvements and fixes#1488
ancazamfir wants to merge 2 commits intocirclefin:mainfrom
ancazamfir:finalize-fixes

Conversation

@ancazamfir
Copy link
Contributor

Closes: #XXX


PR author checklist

Contribution eligibility

  • I am a core contributor, OR I have been explicitly assigned to the linked issue
  • I have read CONTRIBUTING.md and my PR complies with all requirements
  • I understand that PRs not meeting these requirements will be closed without review

For all contributors

For external contributors

@ancazamfir
Copy link
Contributor Author

Closing as we decided to keep the current behavior. We observed that, if an app uses timestamps vs local time to determine if a node is still syncing, aka catchup_threshold, and if catchup_threshold > 1.5 * block_time, the node will never catchup as it gets closer to the tip of the chain.

@ancazamfir ancazamfir closed this Feb 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant