refactor: remove eyre from daft-connect #3719
Merged
Codecov / codecov/project
succeeded
Jan 23, 2025 in 0s
77.86% (-0.05%) compared to bae106c
View this Pull Request on Codecov
77.86% (-0.05%) compared to bae106c
Details
Codecov Report
Attention: Patch coverage is 35.41667% with 186 lines in your changes missing coverage. Please review.
Project coverage is 77.86%. Comparing base (
bae106c) to head (b672f7a).
Report is 3 commits behind head on main.
Additional details and impacted files
@@ Coverage Diff @@
## main #3719 +/- ##
==========================================
- Coverage 77.91% 77.86% -0.05%
==========================================
Files 727 728 +1
Lines 91129 91269 +140
==========================================
+ Hits 71007 71071 +64
- Misses 20122 20198 +76 | Files with missing lines | Coverage Δ | |
|---|---|---|
| src/daft-connect/src/functions.rs | 100.00% <ø> (ø) |
|
| src/daft-connect/src/lib.rs | 94.44% <100.00%> (+1.90%) |
⬆️ |
| src/daft-connect/src/response_builder.rs | 100.00% <100.00%> (ø) |
|
| src/daft-connect/src/execute.rs | 85.91% <96.00%> (+2.52%) |
⬆️ |
| src/daft-connect/src/util.rs | 73.91% <70.00%> (-3.36%) |
⬇️ |
| src/daft-connect/src/connect_service.rs | 59.32% <0.00%> (+0.66%) |
⬆️ |
| src/daft-connect/src/functions/core.rs | 90.27% <50.00%> (-5.38%) |
⬇️ |
| src/daft-connect/src/spark_analyzer/literal.rs | 18.18% <10.00%> (ø) |
|
| src/daft-connect/src/spark_analyzer/datatype.rs | 12.24% <5.00%> (+0.32%) |
⬆️ |
| src/daft-connect/src/display.rs | 54.94% <18.42%> (+0.50%) |
⬆️ |
| ... and 2 more |
Loading