search

filecoin-project / go-f3

Golang implementation of Fast Finality for Filecoin (F3)
Apache License 2.0
11 stars 7 forks source link

Implement WriteAheadLog #640

Closed Kubuxu closed 2 months ago

Kubuxu commented 2 months ago

Pushing the first commit with complete implementation of WAL for peliminary review. Integration in progress.

Kubuxu commented 2 months ago

@masih @Stebalien I would appriciate a light review of the WAL itself, integration is incoming after I fix some issues in it.

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 70.55215% with 48 lines in your changes missing coverage. Please review.

Please upload report for BASE (main@5724153). Learn more about missing BASE report. Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
internal/writeaheadlog/wal.go 69.78% 23 Missing and 19 partials :warning:
f3.go 64.70% 3 Missing and 3 partials :warning:
Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/filecoin-project/go-f3/pull/640/graphs/tree.svg?width=650&height=150&src=pr&token=6uD131t7gs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=filecoin-project)](https://app.codecov.io/gh/filecoin-project/go-f3/pull/640?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=filecoin-project) ```diff @@ Coverage Diff @@ ## main #640 +/- ## ======================================= Coverage ? 79.25% ======================================= Files ? 54 Lines ? 4878 Branches ? 0 ======================================= Hits ? 3866 Misses ? 635 Partials ? 377 ``` | [Files with missing lines](https://app.codecov.io/gh/filecoin-project/go-f3/pull/640?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=filecoin-project) | Coverage Δ | | |---|---|---| | [host.go](https://app.codecov.io/gh/filecoin-project/go-f3/pull/640?src=pr&el=tree&filepath=host.go&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=filecoin-project#diff-aG9zdC5nbw==) | `70.42% <100.00%> (ø)` | | | [wal.go](https://app.codecov.io/gh/filecoin-project/go-f3/pull/640?src=pr&el=tree&filepath=wal.go&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=filecoin-project#diff-d2FsLmdv) | `100.00% <100.00%> (ø)` | | | [f3.go](https://app.codecov.io/gh/filecoin-project/go-f3/pull/640?src=pr&el=tree&filepath=f3.go&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=filecoin-project#diff-ZjMuZ28=) | `71.25% <64.70%> (ø)` | | | [internal/writeaheadlog/wal.go](https://app.codecov.io/gh/filecoin-project/go-f3/pull/640?src=pr&el=tree&filepath=internal%2Fwriteaheadlog%2Fwal.go&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=filecoin-project#diff-aW50ZXJuYWwvd3JpdGVhaGVhZGxvZy93YWwuZ28=) | `69.78% <69.78%> (ø)` | |