Skip to content

Conversation

@OussamaSaoudi
Copy link
Collaborator

@OussamaSaoudi OussamaSaoudi commented Nov 19, 2025

🥞 Stacked PR

Use this link to review incremental changes.


What changes are proposed in this pull request?

This PR introduces serialization and deserialization for the ScanLogReplayProcessor.

How was this change tested?

@OussamaSaoudi OussamaSaoudi changed the title serde first draft feat: Distributed Log Replay serialization/deserialization Nov 19, 2025
@github-actions github-actions bot added the breaking-change Change that require a major version bump label Nov 24, 2025
@OussamaSaoudi OussamaSaoudi force-pushed the stack/dlr_serde branch 6 times, most recently from 0da2ae4 to 274c7bf Compare November 26, 2025 19:13
@codecov
Copy link

codecov bot commented Dec 2, 2025

Codecov Report

❌ Patch coverage is 90.79755% with 60 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.77%. Comparing base (6972292) to head (74cdbd4).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
kernel/src/log_reader/checkpoint_manifest.rs 85.25% 8 Missing and 15 partials ⚠️
kernel/src/distributed/sequential_phase.rs 87.86% 10 Missing and 11 partials ⚠️
kernel/src/log_reader/commit.rs 87.75% 1 Missing and 5 partials ⚠️
kernel/src/scan/log_replay.rs 97.56% 3 Missing and 3 partials ⚠️
kernel/src/utils.rs 85.71% 0 Missing and 3 partials ⚠️
kernel/src/log_segment.rs 50.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1503      +/-   ##
==========================================
+ Coverage   84.55%   84.77%   +0.21%     
==========================================
  Files         123      126       +3     
  Lines       33366    34267     +901     
  Branches    33366    34267     +901     
==========================================
+ Hits        28213    29049     +836     
- Misses       3768     3788      +20     
- Partials     1385     1430      +45     

☔ 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking-change Change that require a major version bump

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant