Skip to content
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

Merged
merged 4 commits into from
Jan 9, 2025

Conversation

Trantorian1
Copy link
Collaborator

Pull Request type

  • Bugfix

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

@Trantorian1 Trantorian1 added bug Report an issue or unexpected behavior sequencer Related to the sequencing logic and implementation labels Dec 30, 2024
@Trantorian1 Trantorian1 self-assigned this Dec 30, 2024
Copy link
Contributor

@Mohiiit Mohiiit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, minor doubts

crates/madara/node/src/main.rs Outdated Show resolved Hide resolved
crates/madara/node/src/main.rs Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
Copy link
Contributor

@notlesh notlesh left a 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.

@Trantorian1
Copy link
Collaborator Author

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.

@antiyro antiyro merged commit 4d1c4b3 into main Jan 9, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Report an issue or unexpected behavior sequencer Related to the sequencing logic and implementation
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

bug(mempool): tick time <= block time
4 participants