You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@datafusion.apache.org by "codecov-commenter (via GitHub)" <gi...@apache.org> on 2026/08/12 20:18:14 UTC

Re: [PR] [branch-55] Backport of refactor(physical-plan): Simplify `ExecutionPlan` API with `replace_children` [datafusion]

codecov-commenter commented on PR #24296:
URL: https://github.com/apache/datafusion/pull/24296#issuecomment-5272331134

   ## [Codecov](https://app.codecov.io/gh/apache/datafusion/pull/24296?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
   :x: Patch coverage is `39.02066%` with `797 lines` in your changes missing coverage. Please review.
   :white_check_mark: Project coverage is 81.14%. Comparing base ([`46bd88a`](https://app.codecov.io/gh/apache/datafusion/commit/46bd88a12538b8115209ad29252ee7aa94985966?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`811b8c8`](https://app.codecov.io/gh/apache/datafusion/commit/811b8c89b00122343033010ec857b03b2e11067e?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
   :warning: Report is 1 commits behind head on branch-55.
   
   | [Files with missing lines](https://app.codecov.io/gh/apache/datafusion/pull/24296?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
   |---|---|---|
   | [datafusion/physical-plan/src/test/exec.rs](https://app.codecov.io/gh/apache/datafusion/pull/24296?src=pr&el=tree&filepath=datafusion%2Fphysical-plan%2Fsrc%2Ftest%2Fexec.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZGF0YWZ1c2lvbi9waHlzaWNhbC1wbGFuL3NyYy90ZXN0L2V4ZWMucnM=) | 0.00% | [60 Missing :warning: ](https://app.codecov.io/gh/apache/datafusion/pull/24296?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [datafusion/physical-plan/src/execution\_plan.rs](https://app.codecov.io/gh/apache/datafusion/pull/24296?src=pr&el=tree&filepath=datafusion%2Fphysical-plan%2Fsrc%2Fexecution_plan.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZGF0YWZ1c2lvbi9waHlzaWNhbC1wbGFuL3NyYy9leGVjdXRpb25fcGxhbi5ycw==) | 50.42% | [58 Missing :warning: ](https://app.codecov.io/gh/apache/datafusion/pull/24296?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [datafusion/core/src/physical\_planner.rs](https://app.codecov.io/gh/apache/datafusion/pull/24296?src=pr&el=tree&filepath=datafusion%2Fcore%2Fsrc%2Fphysical_planner.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZGF0YWZ1c2lvbi9jb3JlL3NyYy9waHlzaWNhbF9wbGFubmVyLnJz) | 36.23% | [44 Missing :warning: ](https://app.codecov.io/gh/apache/datafusion/pull/24296?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [...ysical-plan/src/joins/piecewise\_merge\_join/exec.rs](https://app.codecov.io/gh/apache/datafusion/pull/24296?src=pr&el=tree&filepath=datafusion%2Fphysical-plan%2Fsrc%2Fjoins%2Fpiecewise_merge_join%2Fexec.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZGF0YWZ1c2lvbi9waHlzaWNhbC1wbGFuL3NyYy9qb2lucy9waWVjZXdpc2VfbWVyZ2Vfam9pbi9leGVjLnJz) | 32.00% | [34 Missing :warning: ](https://app.codecov.io/gh/apache/datafusion/pull/24296?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [datafusion/physical-plan/src/aggregates/mod.rs](https://app.codecov.io/gh/apache/datafusion/pull/24296?src=pr&el=tree&filepath=datafusion%2Fphysical-plan%2Fsrc%2Faggregates%2Fmod.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZGF0YWZ1c2lvbi9waHlzaWNhbC1wbGFuL3NyYy9hZ2dyZWdhdGVzL21vZC5ycw==) | 47.36% | [30 Missing :warning: ](https://app.codecov.io/gh/apache/datafusion/pull/24296?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [datafusion/physical-plan/src/sorts/partial\_sort.rs](https://app.codecov.io/gh/apache/datafusion/pull/24296?src=pr&el=tree&filepath=datafusion%2Fphysical-plan%2Fsrc%2Fsorts%2Fpartial_sort.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZGF0YWZ1c2lvbi9waHlzaWNhbC1wbGFuL3NyYy9zb3J0cy9wYXJ0aWFsX3NvcnQucnM=) | 0.00% | [27 Missing :warning: ](https://app.codecov.io/gh/apache/datafusion/pull/24296?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [datafusion/physical-plan/src/buffer.rs](https://app.codecov.io/gh/apache/datafusion/pull/24296?src=pr&el=tree&filepath=datafusion%2Fphysical-plan%2Fsrc%2Fbuffer.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZGF0YWZ1c2lvbi9waHlzaWNhbC1wbGFuL3NyYy9idWZmZXIucnM=) | 0.00% | [26 Missing :warning: ](https://app.codecov.io/gh/apache/datafusion/pull/24296?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [datafusion/physical-plan/src/async\_func.rs](https://app.codecov.io/gh/apache/datafusion/pull/24296?src=pr&el=tree&filepath=datafusion%2Fphysical-plan%2Fsrc%2Fasync_func.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZGF0YWZ1c2lvbi9waHlzaWNhbC1wbGFuL3NyYy9hc3luY19mdW5jLnJz) | 0.00% | [25 Missing :warning: ](https://app.codecov.io/gh/apache/datafusion/pull/24296?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [datafusion/physical-plan/src/limit.rs](https://app.codecov.io/gh/apache/datafusion/pull/24296?src=pr&el=tree&filepath=datafusion%2Fphysical-plan%2Fsrc%2Flimit.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZGF0YWZ1c2lvbi9waHlzaWNhbC1wbGFuL3NyYy9saW1pdC5ycw==) | 61.01% | [23 Missing :warning: ](https://app.codecov.io/gh/apache/datafusion/pull/24296?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [datafusion/physical-plan/src/sorts/sort.rs](https://app.codecov.io/gh/apache/datafusion/pull/24296?src=pr&el=tree&filepath=datafusion%2Fphysical-plan%2Fsrc%2Fsorts%2Fsort.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZGF0YWZ1c2lvbi9waHlzaWNhbC1wbGFuL3NyYy9zb3J0cy9zb3J0LnJz) | 14.81% | [22 Missing and 1 partial :warning: ](https://app.codecov.io/gh/apache/datafusion/pull/24296?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | ... and [41 more](https://app.codecov.io/gh/apache/datafusion/pull/24296?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | |
   
   <details><summary>Additional details and impacted files</summary>
   
   
   
   ```diff
   @@              Coverage Diff              @@
   ##           branch-55   #24296      +/-   ##
   =============================================
   - Coverage      81.29%   81.14%   -0.16%     
   =============================================
     Files           1110     1110              
     Lines         385336   386132     +796     
     Branches      385336   386132     +796     
   =============================================
   + Hits          313258   313318      +60     
   - Misses         53594    54346     +752     
   + Partials       18484    18468      -16     
   ```
   </details>
   
   [:umbrella: View full report in Codecov by Harness](https://app.codecov.io/gh/apache/datafusion/pull/24296?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).   
   :loudspeaker: Have feedback on the report? [Share it here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
   <details><summary> :rocket: New features to boost your workflow: </summary>
   
   - :snowflake: [Test Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, report on failures, and find test suite problems.
   - :package: [JS Bundle Analysis](https://docs.codecov.com/docs/javascript-bundle-analysis): Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
   </details>


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org