You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Similar to #2269. GRANDPA changes are not undone if there's a fork. This could cause consensus issues for functions like handleScheduledChange() and handleForcedChange() and handleGrandpaChangesOnImport() if different changes are scheduled in the forks then we will either error or return nil for the second block that is processed. In Substrate they use a ForkTree for the schedule changes and a Vec for the forced changes as seen in this struct from substrate/client/finality-grandpa/src/authorities.rs.
The text was updated successfully, but these errors were encountered:
danforbes
changed the title
Track GRANDPA Scheduled Changes Across Forks
Track GRANDPA Changes Across Forks
Mar 7, 2022
Similar to #2269. GRANDPA changes are not undone if there's a fork. This could cause consensus issues for functions like
handleScheduledChange()
andhandleForcedChange()
andhandleGrandpaChangesOnImport()
if different changes are scheduled in the forks then we will either error or returnnil
for the second block that is processed. In Substrate they use aForkTree
for the schedule changes and aVec
for the forced changes as seen in this struct fromsubstrate/client/finality-grandpa/src/authorities.rs
.The text was updated successfully, but these errors were encountered: