Skip to content

fix: nested runtimes in stream adapter#4148

Merged
ion-elgreco merged 1 commit intodelta-io:mainfrom
ion-elgreco:fix/nested-runtimes-stream-adapter
Jan 30, 2026
Merged

fix: nested runtimes in stream adapter#4148
ion-elgreco merged 1 commit intodelta-io:mainfrom
ion-elgreco:fix/nested-runtimes-stream-adapter

Conversation

@ion-elgreco
Copy link
Copy Markdown
Collaborator

@ion-elgreco ion-elgreco marked this pull request as ready for review January 30, 2026 14:11
@github-actions github-actions Bot added the binding/python Issues for the Python package label Jan 30, 2026
@ion-elgreco ion-elgreco force-pushed the fix/nested-runtimes-stream-adapter branch from 569a2dd to fa4df1a Compare January 30, 2026 14:11
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 30, 2026

Codecov Report

❌ Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.22%. Comparing base (1612ba1) to head (a74dc3c).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
python/src/reader.rs 0.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4148      +/-   ##
==========================================
- Coverage   76.24%   76.22%   -0.02%     
==========================================
  Files         164      164              
  Lines       45806    45785      -21     
  Branches    45806    45785      -21     
==========================================
- Hits        34924    34901      -23     
- Misses       9209     9212       +3     
+ Partials     1673     1672       -1     

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

Copy link
Copy Markdown
Collaborator

@roeap roeap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i think there was a similar error in the kernel library as well, runtime juggling is just another gift that keeps on giving 🤣.

@ion-elgreco ion-elgreco enabled auto-merge (squash) January 30, 2026 14:32
@ion-elgreco ion-elgreco force-pushed the fix/nested-runtimes-stream-adapter branch from fa4df1a to a74dc3c Compare January 30, 2026 14:32
@ion-elgreco ion-elgreco merged commit c095787 into delta-io:main Jan 30, 2026
28 of 29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

binding/python Issues for the Python package

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Bug Report: Nested Tokio Runtime Panic in write_deltalake

2 participants