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

Allow copy constructor for raft_params #10

Merged
merged 1 commit into from
Aug 8, 2019
Merged

Conversation

greensky00
Copy link
Contributor

  • raft_params is just a simple structure and there is no reason to
    block copy operation.

  • Modified related APIs.

* `raft_params` is just a simple structure and there is no reason to
block copy operation.

* Modified related APIs.
@codecov-io
Copy link

Codecov Report

Merging #10 into master will decrease coverage by 0.412%.
The diff coverage is 87.5%.

@@              Coverage Diff              @@
##            master       #10       +/-   ##
=============================================
- Coverage   82.458%   82.045%   -0.413%     
=============================================
  Files           56        56               
  Lines         4019      3999       -20     
=============================================
- Hits          3314      3281       -33     
- Misses         705       718       +13

@greensky00 greensky00 merged commit 84ecf5c into eBay:master Aug 8, 2019
JackyWoo pushed a commit to JackyWoo/NuRaft that referenced this pull request Sep 30, 2022
xiaoxichen pushed a commit to xiaoxichen/NuRaft that referenced this pull request Jan 16, 2024
xiaoxichen pushed a commit to xiaoxichen/NuRaft that referenced this pull request Jan 16, 2024
completion cb is part of rpc data
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants