-
Notifications
You must be signed in to change notification settings - Fork 50
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
fix(block_time): block time can no longer be less than pending block update time #447
Conversation
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.
lgtm, minor doubts
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.
Left a couple nits/questions, but also wanted to ask if block time
and bending block update time
are documented anywhere. I don't see them in --help
.
Tick time
was also confusing to me while reading through the PR/issue, is that a user-facing term? If not, no big deal.
Yep, this was pretty confusing to me too when I started work on this PR. I have added docs to all of this into the CLI. Hopefully this makes this part of the code a bit clearer. |
Pull Request type
What is the current behavior?
Block time can be less than pending block update time (also known as tick time). This is an issue as it means only empty blocks will be produced.
Resolves: #277
What is the new behavior?
Forbids the node from starting as a sequencer if ever its block time is less than its tick time.
Does this introduce a breaking change?
No