Skip to content

chore: copy corrupted WAL and docs to .broken dir#395

Merged
cheb0 merged 4 commits into
mainfrom
0-store-corrupted-wal-files
May 12, 2026
Merged

chore: copy corrupted WAL and docs to .broken dir#395
cheb0 merged 4 commits into
mainfrom
0-store-corrupted-wal-files

Conversation

@cheb0
Copy link
Copy Markdown
Collaborator

@cheb0 cheb0 commented Apr 2, 2026

Description

If WAL file is corrupted, then increment a metric and copy to a separate directory .broken


  • I have read and followed all requirements in CONTRIBUTING.md;
  • I used LLM/AI assistance to make this pull request;

@cheb0 cheb0 requested a review from eguguchkin April 2, 2026 06:28
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 2, 2026

Codecov Report

❌ Patch coverage is 19.04762% with 34 lines in your changes missing coverage. Please review.
✅ Project coverage is 70.60%. Comparing base (0e194d8) to head (6ee8a1a).

Files with missing lines Patch % Lines
frac/active.go 8.69% 20 Missing and 1 partial ⚠️
util/fs.go 0.00% 11 Missing ⚠️
storage/wal_reader.go 75.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #395      +/-   ##
==========================================
- Coverage   70.73%   70.60%   -0.14%     
==========================================
  Files         219      219              
  Lines       16930    16967      +37     
==========================================
+ Hits        11976    11979       +3     
- Misses       4059     4091      +32     
- Partials      895      897       +2     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Comment thread util/fs.go Outdated
Comment thread util/fs.go
@eguguchkin eguguchkin requested review from eguguchkin and moflotas and removed request for eguguchkin April 6, 2026 10:26
@eguguchkin eguguchkin added this to the v0.71.0 milestone Apr 13, 2026
@cheb0 cheb0 merged commit e0cf437 into main May 12, 2026
8 checks passed
@cheb0 cheb0 deleted the 0-store-corrupted-wal-files branch May 12, 2026 07:01
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.

5 participants