Skip to content

Commit

Permalink
[consensus] reduce leader timeout to 200ms (#21446)
Browse files Browse the repository at this point in the history
## Description 

With round interval of 70-80ms, using 250ms for leader timeout seems a
bit excessive.

## Test plan 

PT

---

## Release notes

Check each box that your changes affect. If none of the boxes relate to
your changes, release notes aren't required.

For each box you select, include information after the relevant heading
that describes the impact of your changes that a user might notice and
any actions they must take to implement updates.

- [ ] Protocol: 
- [ ] Nodes (Validators and Full nodes): 
- [ ] gRPC:
- [ ] JSON-RPC: 
- [ ] GraphQL: 
- [ ] CLI: 
- [ ] Rust SDK:
  • Loading branch information
mwtian authored Mar 10, 2025
1 parent 80e680b commit 69e76ed
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion consensus/config/src/parameters.rs
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ pub struct Parameters {

impl Parameters {
pub(crate) fn default_leader_timeout() -> Duration {
Duration::from_millis(250)
Duration::from_millis(200)
}

pub(crate) fn default_min_round_delay() -> Duration {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ expression: parameters
---
leader_timeout:
secs: 0
nanos: 250000000
nanos: 200000000
min_round_delay:
secs: 0
nanos: 50000000
Expand Down

0 comments on commit 69e76ed

Please sign in to comment.