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

Reduce a series of default_timeout_ms for RaftKeeper #272

Merged
merged 4 commits into from
May 22, 2024

Conversation

lzydmxy
Copy link
Contributor

@lzydmxy lzydmxy commented May 21, 2024

Which issues of this PR fixes:

This PR try to fix #255

Change log:

  1. Reduce default_election_timeout_ms from [10000, 20000] to [3000, 5000]
  2. Reduce default_session_timeout_ms from [10000, 100000] to [1000, 3600000]
  3. Reduce default_operation_timeout_ms from 10000 to 3000

@JackyWoo
Copy link
Contributor

operation_time and min_session_time should also be reduced.

@lzydmxy lzydmxy changed the title Reduce default_election_timeout_ms from [10000, 20000] to [3000, 5000] Reduce a series of default_timeout_ms for RaftKeeper May 21, 2024
@JackyWoo
Copy link
Contributor

LGTM

@JackyWoo JackyWoo merged commit ccea3ad into JDRaftKeeper:master May 22, 2024
6 of 7 checks passed
@JackyWoo JackyWoo added this to the Release v2.0.5 milestone May 22, 2024
@JackyWoo JackyWoo added documentation Improvements or additions to documentation enhancement Enhancements and removed documentation Improvements or additions to documentation labels May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reduce the default timeout value since we now support async snapshot
2 participants