Skip to content

fix: disk durability improvements#28

Merged
choudlet merged 1 commit into
mainfrom
disk-durability
Apr 22, 2026
Merged

fix: disk durability improvements#28
choudlet merged 1 commit into
mainfrom
disk-durability

Conversation

@choudlet

Copy link
Copy Markdown
Collaborator

Disk Durability Improvements

@github-actions

Copy link
Copy Markdown

Test Results

 60 files  ± 0   60 suites  ±0   1m 8s ⏱️ -2s
486 tests + 6  486 ✅ + 6  0 💤 ±0  0 ❌ ±0 
972 runs  +12  972 ✅ +12  0 💤 ±0  0 ❌ ±0 

Results for commit c83993a. ± Comparison against base commit 943b006.

This pull request removes 3 and adds 9 tests. Note that renamed tests count towards both.
com.metarouter.analytics.storage.EventDiskStoreTest ‑ read returns null for corrupted file and deletes it
com.metarouter.analytics.storage.EventDiskStoreTest ‑ read returns null for unrecognized schema version and deletes file
com.metarouter.analytics.storage.EventDiskStoreTest ‑ read returns null when version field is missing
com.metarouter.analytics.queue.PersistableEventQueueTest ‑ clear wipes pendingOverflow too
com.metarouter.analytics.queue.PersistableEventQueueTest ‑ pendingOverflow preserves events when disk write fails at capacity
com.metarouter.analytics.queue.PersistableEventQueueTest ‑ pendingOverflow retained across repeated failures
com.metarouter.analytics.storage.EventDiskStoreTest ‑ read returns null only when file is absent
com.metarouter.analytics.storage.EventDiskStoreTest ‑ read throws on corrupted file and still deletes it
com.metarouter.analytics.storage.EventDiskStoreTest ‑ read throws on real I-O failure
com.metarouter.analytics.storage.EventDiskStoreTest ‑ read throws on unrecognized schema version and still deletes file
com.metarouter.analytics.storage.EventDiskStoreTest ‑ read throws when version field is missing and still deletes file
com.metarouter.analytics.storage.EventDiskStoreTest ‑ write throws on real I-O failure

@choudlet choudlet merged commit 50b2102 into main Apr 22, 2026
3 checks passed
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.

1 participant