feat: use raft-engine crate to reimplement logstore#799
Merged
v0y4g3r merged 19 commits intoGreptimeTeam:developfrom Jan 5, 2023
Merged
feat: use raft-engine crate to reimplement logstore#799v0y4g3r merged 19 commits intoGreptimeTeam:developfrom
v0y4g3r merged 19 commits intoGreptimeTeam:developfrom
Conversation
evenyag
reviewed
Dec 29, 2022
v0y4g3r
force-pushed
the
feat/raft-engine-logstore
branch
5 times, most recently
from
0fa96ed to
cc50ccc
Compare
v0y4g3r
force-pushed
the
feat/raft-engine-logstore
branch
from
0aa143d to
03a7093
Compare
Codecov Report
@@ Coverage Diff @@
## develop #799 +/- ##
===========================================
- Coverage 85.95% 85.89% -0.06%
===========================================
Files 421 416 -5
Lines 55976 55160 -816
===========================================
- Hits 48114 47381 -733
+ Misses 7862 7779 -83
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
killme2008
reviewed
Jan 4, 2023
v0y4g3r
force-pushed
the
feat/raft-engine-logstore
branch
2 times, most recently
from
e1afede to
fdba708
Compare
evenyag
reviewed
Jan 4, 2023
fengjiachun
reviewed
Jan 5, 2023
… entry and namespace
v0y4g3r
force-pushed
the
feat/raft-engine-logstore
branch
from
58f30df to
2de59ff
Compare
fengjiachun
reviewed
Jan 5, 2023
waynexia
reviewed
Jan 5, 2023
evenyag
approved these changes
Jan 5, 2023
Contributor
Author
|
@waynexia PTAL |
This was referenced Jan 6, 2023
Closed
v0y4g3r
added a commit
that referenced
this pull request
Jan 6, 2023
(cherry picked from commit 8f5ecef)
paomian
pushed a commit
to paomian/greptimedb
that referenced
this pull request
Oct 19, 2023
* chore: remove useless method in Entry trait, add proto definition for entry and namespace * feat: add proto definition for raft-engine based logstore * feat: introduce RaftEngineLogstore * feat: impl read for raft engine log store * feat: impl raft engine logstore * feat: raft engine logstore start and stop * feat: add purge bg task * fix: license header * fix: clippy * fix: toml files * feat: add some test cases * fix: CR comments * fix: CR comments * fix: check namespace validity and state of logstore * fix: CR comments; add config item to control sync/async flush per write * fix: remove unused error variants * fix: unit tests * fix: use compare and exchange to stop logstore * fix: CR comments
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
LocalFileLogstoreChecklist
Refer to a related PR or issue link (optional)
#793