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

feat: implement curp wal storage #687

Merged
merged 1 commit into from
Jul 9, 2024

Conversation

bsbds
Copy link
Collaborator

@bsbds bsbds commented Mar 11, 2024

Depends-On: #740

Please briefly answer these questions:

  • what problem are you trying to solve? (or if there's no problem, what's the motivation for this change?)

    This is part of the wal implementation, which implements the WAL storage type

  • what changes does this pull request make?

  • are there any non-obvious implications of these changes? (does it break compatibility with previous versions, etc)

@bsbds bsbds force-pushed the curp-wal-storage branch 2 times, most recently from 71d1c79 to 25af141 Compare March 11, 2024 02:24
@bsbds bsbds force-pushed the curp-wal-storage branch 2 times, most recently from bd42b0f to 0b5505d Compare March 26, 2024 02:54
@bsbds bsbds marked this pull request as ready for review March 26, 2024 02:59
Copy link

mergify bot commented Apr 18, 2024

@bsbds Convert your pr to draft since CI failed

@mergify mergify bot marked this pull request as draft April 18, 2024 06:52
@mergify mergify bot requested a review from a team April 19, 2024 04:36
@bsbds bsbds force-pushed the curp-wal-storage branch from 0b5505d to d87d8bf Compare April 19, 2024 07:24
Copy link

mergify bot commented Apr 19, 2024

@bsbds Convert your pr to draft since CI failed

@bsbds bsbds force-pushed the curp-wal-storage branch from d87d8bf to 17635fe Compare April 19, 2024 07:37
Copy link

mergify bot commented Apr 19, 2024

@bsbds Convert your pr to draft since CI failed

@bsbds bsbds force-pushed the curp-wal-storage branch from 17635fe to c197b13 Compare April 19, 2024 07:39
Copy link

mergify bot commented Apr 19, 2024

@bsbds Convert your pr to draft since CI failed

@bsbds bsbds force-pushed the curp-wal-storage branch from c197b13 to 28100b4 Compare April 19, 2024 09:55
Copy link

mergify bot commented Apr 19, 2024

@bsbds Convert your pr to draft since CI failed

@bsbds bsbds force-pushed the curp-wal-storage branch from 28100b4 to fd5d77c Compare April 19, 2024 10:37
@bsbds bsbds marked this pull request as ready for review April 19, 2024 10:38
Copy link

mergify bot commented Apr 19, 2024

@bsbds Convert your pr to draft since CI failed

@mergify mergify bot marked this pull request as draft April 19, 2024 10:47
@bsbds bsbds force-pushed the curp-wal-storage branch from fd5d77c to 25ca007 Compare April 19, 2024 12:36
@bsbds bsbds marked this pull request as ready for review April 22, 2024 07:07
@Phoenix500526
Copy link
Collaborator

@Mergifyio rebase

Copy link

mergify bot commented Apr 29, 2024

rebase

✅ Branch has been successfully rebased

Copy link

mergify bot commented May 9, 2024

@bsbds Your PR is in conflict and cannot be merged.

@bsbds bsbds force-pushed the curp-wal-storage branch from ea1b72d to 1d1c293 Compare May 11, 2024 02:15
Copy link

codecov bot commented May 11, 2024

Codecov Report

Attention: Patch coverage is 80.45977% with 68 lines in your changes missing coverage. Please review.

Project coverage is 75.73%. Comparing base (e35b35a) to head (c92ec80).
Report is 132 commits behind head on master.

Files Patch % Lines
crates/curp/src/server/storage/wal/mod.rs 73.41% 20 Missing and 22 partials ⚠️
crates/curp/src/server/storage/wal/test_util.rs 71.42% 16 Missing ⚠️
crates/curp/src/server/storage/wal/error.rs 0.00% 6 Missing ⚠️
crates/curp/src/server/storage/wal/tests.rs 97.39% 3 Missing ⚠️
crates/curp/src/server/storage/wal/config.rs 92.30% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #687      +/-   ##
==========================================
+ Coverage   75.55%   75.73%   +0.17%     
==========================================
  Files         180      192      +12     
  Lines       26938    28121    +1183     
  Branches    26938    28121    +1183     
==========================================
+ Hits        20353    21297     +944     
- Misses       5366     5506     +140     
- Partials     1219     1318      +99     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mergify mergify bot requested a review from a team June 6, 2024 02:14
@mergify mergify bot requested a review from a team June 6, 2024 02:14
@mergify mergify bot requested a review from a team June 6, 2024 02:18
@mergify mergify bot requested a review from a team June 6, 2024 02:33
@mergify mergify bot requested a review from a team June 6, 2024 02:35
@mergify mergify bot requested a review from a team June 13, 2024 04:01
This was referenced Jun 16, 2024
@bsbds bsbds force-pushed the curp-wal-storage branch from ca65995 to b9ce399 Compare June 21, 2024 02:50
Copy link
Collaborator

@Phoenix500526 Phoenix500526 left a comment

Choose a reason for hiding this comment

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

Please squash these commits into one

@mergify mergify bot requested a review from a team June 24, 2024 06:24
Signed-off-by: bsbds <[email protected]>

refactor: wal storage

Signed-off-by: bsbds <[email protected]>
@bsbds bsbds force-pushed the curp-wal-storage branch from b9ce399 to c92ec80 Compare June 25, 2024 01:09
@bsbds bsbds added this to the v0.7.0 milestone Jul 1, 2024
Copy link
Collaborator

@Phoenix500526 Phoenix500526 left a comment

Choose a reason for hiding this comment

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

LGTM

@mergify mergify bot requested a review from a team July 9, 2024 02:24
@Phoenix500526 Phoenix500526 merged commit 8ffa516 into xline-kv:master Jul 9, 2024
14 of 16 checks passed
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.

2 participants