-
Notifications
You must be signed in to change notification settings - Fork 18
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.
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
feat: set finalized and safe hash (parlia) #54
feat: set finalized and safe hash (parlia) #54
Conversation
b7fab30
to
bd16a0b
Compare
crates/bsc/tasks/src/task.rs
Outdated
error!(target: "consensus::parlia", ?fcu_response, "Forkchoice update returned invalid response"); | ||
break | ||
} | ||
ForkchoiceStatus::Syncing => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if it is ForkchoiceStatus::Syncing
, why both to send the update in the loop again?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because blocktree will return the fcu status code, the function here is to repeatedly send fcu until the Valid
is returned.
56da573
to
bee8093
Compare
4b29bbb
to
f2a0a5a
Compare
f2a0a5a
to
e247252
Compare
Description
feat: set finalized and safe hash (parlia)
Rationale
Notifying
BlockchainTree
of finalized and safe hash can finalize the blocks, and rpc can query it using thefinalized
andsafe
tags.Example
n/a
Changes
Notable changes:
Potential Impacts
no