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.

Sign up for GitHub

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

Jump to bottom

Remove raftkeeper 1.0 only code #352

Merged
merged 1 commit into from
Sep 9, 2024
Merged

Remove raftkeeper 1.0 only code #352

merged 1 commit into from
Sep 9, 2024

Conversation

JackyWoo
Copy link
Contributor

@JackyWoo JackyWoo commented Sep 6, 2024 •
edited
Loading

Which issues of this PR fixes:

Fix #351, Fix #266

Change log:

  1. Remove raftkeeper 1.0 only code
  2. Use steady_lock instead of system_lock

@JackyWoo JackyWoo added the code refactor Code refactor label Sep 6, 2024
@JackyWoo JackyWoo added this to the Release v2.2.0 milestone Sep 6, 2024
@JackyWoo JackyWoo mentioned this pull request Sep 6, 2024
Closed
@JackyWoo JackyWoo added the performance Performance promotion label Sep 7, 2024
@lzydmxy
Copy link
Contributor

lzydmxy commented Sep 9, 2024

LGTM

@lzydmxy lzydmxy merged commit 78c8285 into JDRaftKeeper:master Sep 9, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lzydmxy lzydmxy lzydmxy approved these changes

Assignees

@lzydmxy lzydmxy

Labels
code refactor Code refactor performance Performance promotion
Projects
None yet
Milestone
Release v2.2.0
Development

Successfully merging this pull request may close these issues.

Remove code for raftkeeper 1.0 Using std::chrono::steady_clock instead of system_clock in RaftKeeper
2 participants
@JackyWoo @lzydmxy