Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Building Storage #33

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
Merged
merged 94 commits into from
Apr 23, 2025
Merged

Building Storage #33

merged 94 commits into from
Apr 23, 2025

Conversation

@jathurchan
Copy link
Owner

@jathurchan jathurchan commented Apr 3, 2025 •
edited
Loading

closes #32

jathurchan added 30 commits March 18, 2025 19:02
@jathurchan
Defined State for servers in the Raft cluster
f5b10c6
@jathurchan
Merge pull request #15 from jathurchan/9-implement-raft-state
a487710
@jathurchan
Renamed State to RaftState
2a720f1
@jathurchan
Renamed RaftState to RaftRole
52e2eed
@jathurchan
feat: Defined Storage interface
c2d8dfe
@jathurchan
feat: Implemented FileStorage
0b97ac2
@jathurchan
feat: Implement CI
8cd10fd
@jathurchan
feat: added code coverage badge
c64c62b
@jathurchan
Removed .gitignore
a180b55
@jathurchan
Merge pull request #20 from jathurchan/18-implement-storage
72e05b2 
Feat: Design & Implementation of Storage
@jathurchan
feat: Added .gitignore
5ec46a4
@jathurchan
Merge pull request #25 from jathurchan/24-add-gitignore
d983bad 
feat: Added .gitignore
@jathurchan
feat: Added InstallSnapshot RPC to raft.proto
d438982
@jathurchan
feat: Implemented RaftOptions
6321b7e
@jathurchan
fix: Improved Validate() to achieve 100% test coverage
d47b5f2
@jathurchan
feat: Introduced RaftConfig to enforce options validation
66e85f0
@jathurchan
Merge pull request #27 from jathurchan/26-implement-options
8310581 
Implement RaftOptions
@jathurchan
feat: Defined Logger interface
2291bdf
@jathurchan
feat: Implemented NoOpLogger
d2503a0
@jathurchan
feat: Implemented StdLogger
01904e2
@jathurchan
fix: Added usage to Logger comments
68fc32f
@jathurchan
fix: Added unit tests and fixed Logger implementations
284bd1c
@jathurchan
fix: Improved test coverage for NoopLogger Fatalw
22012b1
@jathurchan
Merge pull request #30 from jathurchan/28-introduce-logger
22ad003 
feat: Introduce `Logger`
@jathurchan
feat: Updated Config for Raft
5f4ce72
@jathurchan
feat: Introduced feature flags, Tuning parameters...
0f67b20
@jathurchan
fix: Updated Storage and removed outdated implementation
e8f3596
@jathurchan
feat: Defined core structs and constants for storage
c94c44c
@jathurchan
Introduced new packages for storage and types
31764e9
@jathurchan
Introduced new package for logger
e6b5c2b
jathurchan added 16 commits April 18, 2025 17:43
@jathurchan
fix: Fixed readChunks & Added tests for helpers (100%)
15fc8f3
@jathurchan
fix: Fixed ScanRange and added tests for logEntryReader (100%)
9807fff
@jathurchan
fix: Removed duplicate FailAndRollback
4eaf0e6
@jathurchan
fix: Added tests for writer.go (100%)
0124917
@jathurchan
feat: Fixed indexService methods and added tests (100%)
9a62558
@jathurchan
fix: Improved rwOperationLocker for testing and added tests (100%)
0f6c8aa
@jathurchan
fix: Added tests for snapshotWriter, snapshotReader (100%)
c8e6bec
@jathurchan
fix: Added tests to recoveryService (100%)
309d8e1
@jathurchan
fix: Removed deadlock
0ce657c
@jathurchan
feat: Added tests for file storage (75.3%)
7a28dbb
@jathurchan
feat: Added helpers and corresponding tests (100%)
4eae817
@jathurchan
feat: Introduced metrics and added tests (100%)
0729383
@jathurchan
fix: Added missing read_entries
5c9adf0
@jathurchan
fix: Improved test naming
dd1bec9
@jathurchan
feat: Integrated metrics and some fixes
3244f16
@jathurchan
feat: Implemented tests for storage (overall 96.5%)
b107d25
@jathurchan jathurchan mentioned this pull request Apr 23, 2025
Open
mathusanm6

This comment was marked as duplicate.

Sign in to view

mathusanm6
mathusanm6 approved these changes Apr 23, 2025
View reviewed changes
Copy link
Collaborator

@mathusanm6 mathusanm6 left a comment

Choose a reason for hiding this comment

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

LGTM!

@jathurchan jathurchan changed the base branch from 9-implement-raft to main April 23, 2025 18:55
@jathurchan jathurchan linked an issue Apr 23, 2025 that may be closed by this pull request
Build Storage Layer #32
Closed
4 tasks
@jathurchan jathurchan marked this pull request as ready for review April 23, 2025 19:06
@jathurchan jathurchan merged commit 1211044 into main Apr 23, 2025
2 checks passed
@jathurchan jathurchan deleted the 32-build-storage-layer branch May 22, 2025 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mathusanm6 mathusanm6 mathusanm6 approved these changes

@Patrice-Zhou Patrice-Zhou Awaiting requested review from Patrice-Zhou

Assignees

@jathurchan jathurchan

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Build Storage Layer

3 participants

@jathurchan @codecov-commenter @mathusanm6