Skip to content

Conversation

@mbutrovich
Copy link
Contributor

Which issue does this PR close?

Closes #.

Rationale for this change

What changes are included in this PR?

How are these changes tested?

@mbutrovich mbutrovich changed the title perf: [WIP] [iceberg] Per partition FileScanTasks perf: [WIP] [iceberg] Per-partition FileScanTasks Jan 27, 2026
@codecov-commenter
Copy link

codecov-commenter commented Jan 27, 2026

Codecov Report

❌ Patch coverage is 86.21908% with 39 lines in your changes missing coverage. Please review.
✅ Project coverage is 60.09%. Comparing base (f09f8af) to head (89c780c).
⚠️ Report is 901 commits behind head on main.

Files with missing lines Patch % Lines
.../comet/serde/operator/CometIcebergNativeScan.scala 86.97% 21 Missing and 4 partials ⚠️
...cala/org/apache/spark/sql/comet/CometExecRDD.scala 78.18% 9 Missing and 3 partials ⚠️
...ain/scala/org/apache/comet/CometExecIterator.scala 75.00% 1 Missing ⚠️
...n/scala/org/apache/spark/sql/comet/operators.scala 96.87% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #3297      +/-   ##
============================================
+ Coverage     56.12%   60.09%   +3.97%     
- Complexity      976     1462     +486     
============================================
  Files           119      175      +56     
  Lines         11743    16268    +4525     
  Branches       2251     2690     +439     
============================================
+ Hits           6591     9777    +3186     
- Misses         4012     5138    +1126     
- Partials       1140     1353     +213     

☔ 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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants