Skip to content
This repository was archived by the owner on Jan 20, 2026. It is now read-only.

Fix async flush logic and flaky unit test#122

Merged
yzang2019 merged 5 commits into
mainfrom
yzang/fix-async-flush
Oct 9, 2025
Merged

Fix async flush logic and flaky unit test#122
yzang2019 merged 5 commits into
mainfrom
yzang/fix-async-flush

Address comment

42ec3bc
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project failed Oct 9, 2025 in 1s

40.76% (target 60.00%)

View this Pull Request on Codecov

40.76% (target 60.00%)

Details

Codecov Report

❌ Patch coverage is 65.38462% with 18 lines in your changes missing coverage. Please review.
✅ Project coverage is 40.76%. Comparing base (c3ee918) to head (42ec3bc).
⚠️ Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
ss/store.go 11.76% 12 Missing and 3 partials ⚠️
ss/pebbledb/db.go 85.71% 2 Missing and 1 partial ⚠️

❌ Your project check has failed because the head coverage (40.76%) is below the target coverage (60.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #122      +/-   ##
==========================================
+ Coverage   40.00%   40.76%   +0.76%     
==========================================
  Files          59       59              
  Lines        7510     7534      +24     
==========================================
+ Hits         3004     3071      +67     
+ Misses       4161     4119      -42     
+ Partials      345      344       -1     
Files with missing lines Coverage Δ
config/config.go 0.00% <ø> (ø)
ss/rocksdb/db.go 56.60% <100.00%> (+11.58%) ⬆️
ss/pebbledb/db.go 61.11% <85.71%> (+5.67%) ⬆️
ss/store.go 36.76% <11.76%> (-11.13%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.