Skip to content

WIP: Coroutines for snapshot builds and commit path - #3315

Draft
scovich wants to merge 2 commits into
delta-io:mainfrom
scovich:kernel-coroutines-prototype-v3
Draft

WIP: Coroutines for snapshot builds and commit path#3315
scovich wants to merge 2 commits into
delta-io:mainfrom
scovich:kernel-coroutines-prototype-v3

Conversation

@scovich

@scovich scovich commented Sep 11, 2026

Copy link
Copy Markdown
Collaborator

What changes are proposed in this pull request?

How was this change tested?

@github-actions

Copy link
Copy Markdown

PR title does not match the required pattern. Please ensure you follow the conventional commits spec.

Your title should start with feat:, fix:, chore:, docs:, perf:, refactor:, test:, or ci:, and if it's a breaking change that should be suffixed with a ! (like feat!:), and then a 1-72 character brief description of your change.

Title: WIP: Coroutines for snapshot builds and commit path

@github-actions github-actions Bot added the breaking-change Public API change that could cause downstream compilation failures. Requires a major version bump. label Sep 11, 2026
@codecov

codecov Bot commented Sep 11, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 82.42489% with 661 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.10%. Comparing base (e315ecf) to head (e1ae043).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
default-engine/src/coroutine.rs 67.18% 122 Missing and 26 partials ⚠️
kernel/src/coroutine/engine.rs 80.89% 67 Missing and 10 partials ⚠️
kernel/src/transaction/mod.rs 78.75% 10 Missing and 65 partials ⚠️
kernel/src/coroutine/kernel/channel.rs 73.83% 56 Missing ⚠️
delta-kernel-unity-catalog/src/committer.rs 75.00% 27 Missing and 8 partials ⚠️
kernel/src/coroutine/core.rs 79.28% 20 Missing and 15 partials ⚠️
kernel/src/coroutine/kernel/mod.rs 78.91% 29 Missing and 6 partials ⚠️
kernel/src/log_segment/mod.rs 92.02% 6 Missing and 20 partials ⚠️
kernel/src/snapshot/mod.rs 91.30% 3 Missing and 19 partials ⚠️
kernel/src/transaction/update.rs 78.46% 0 Missing and 14 partials ⚠️
... and 28 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3315      +/-   ##
==========================================
- Coverage   90.34%   90.10%   -0.25%     
==========================================
  Files         250      260      +10     
  Lines       89131    92071    +2940     
  Branches    89131    92071    +2940     
==========================================
+ Hits        80524    82956    +2432     
- Misses       5672     6015     +343     
- Partials     2935     3100     +165     

☔ View full report in Codecov by Harness.
📢 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 Public API change that could cause downstream compilation failures. Requires a major version bump.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant