Skip to content

fix: capture S3 diagnostics and stop logging presigned URLs on SQL cache failures - #114

Draft
tkislan wants to merge 1 commit into
mainfrom
fix/113-sql-cache-upload-diagnostics
Draft

fix: capture S3 diagnostics and stop logging presigned URLs on SQL cache failures#114
tkislan wants to merge 1 commit into
mainfrom
fix/113-sql-cache-upload-diagnostics

fix: capture S3 diagnostics and stop logging presigned URLs on SQL ca…

c9fa363
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Jul 29, 2026 in 0s

74.85% (+0.43%) compared to 6198101

View this Pull Request on Codecov

74.85% (+0.43%) compared to 6198101

Details

Codecov Report

❌ Patch coverage is 94.49541% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.85%. Comparing base (6198101) to head (c9fa363).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
deepnote_toolkit/sql/sql_caching.py 96.19% 3 Missing and 1 partial ⚠️
deepnote_toolkit/sql/sql_execution.py 50.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #114      +/-   ##
==========================================
+ Coverage   74.42%   74.85%   +0.43%     
==========================================
  Files          95       95              
  Lines        5704     5790      +86     
  Branches      850      860      +10     
==========================================
+ Hits         4245     4334      +89     
+ Misses       1182     1177       -5     
- Partials      277      279       +2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.