codecov-commenter commented on PR #27243: URL: https://github.com/apache/datafusion/pull/27243#issuecomment-6724881576
## [Codecov](https://app.codecov.io/gh/apache/datafusion/pull/27243?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 `91.48311%` with `58 lines` in your changes missing coverage. Please review. :white_check_mark: Project coverage is 83.28%. Comparing base ([`766e259`](https://app.codecov.io/gh/apache/datafusion/commit/766e259d7ed78abf6ec108a1c6b7a26e9eb7a91a?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`5fb670c`](https://app.codecov.io/gh/apache/datafusion/commit/5fb670cb0c2456b7dc215ae1d1a4ec174534079b?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)). | [Files with missing lines](https://app.codecov.io/gh/apache/datafusion/pull/27243?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines | |---|---|---| | [...usion/physical-plan/src/spill/spilled\_row\_sizes.rs](https://app.codecov.io/gh/apache/datafusion/pull/27243?src=pr&el=tree&filepath=datafusion%2Fphysical-plan%2Fsrc%2Fspill%2Fspilled_row_sizes.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZGF0YWZ1c2lvbi9waHlzaWNhbC1wbGFuL3NyYy9zcGlsbC9zcGlsbGVkX3Jvd19zaXplcy5ycw==) | 92.72% | [19 Missing and 4 partials :warning: ](https://app.codecov.io/gh/apache/datafusion/pull/27243?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | [datafusion/physical-plan/src/spill/mod.rs](https://app.codecov.io/gh/apache/datafusion/pull/27243?src=pr&el=tree&filepath=datafusion%2Fphysical-plan%2Fsrc%2Fspill%2Fmod.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZGF0YWZ1c2lvbi9waHlzaWNhbC1wbGFuL3NyYy9zcGlsbC9tb2QucnM=) | 87.61% | [1 Missing and 13 partials :warning: ](https://app.codecov.io/gh/apache/datafusion/pull/27243?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/stream.rs](https://app.codecov.io/gh/apache/datafusion/pull/27243?src=pr&el=tree&filepath=datafusion%2Fphysical-plan%2Fsrc%2Fsorts%2Fstream.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZGF0YWZ1c2lvbi9waHlzaWNhbC1wbGFuL3NyYy9zb3J0cy9zdHJlYW0ucnM=) | 90.09% | [5 Missing and 6 partials :warning: ](https://app.codecov.io/gh/apache/datafusion/pull/27243?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | [...afusion/physical-plan/src/sorts/streaming\_merge.rs](https://app.codecov.io/gh/apache/datafusion/pull/27243?src=pr&el=tree&filepath=datafusion%2Fphysical-plan%2Fsrc%2Fsorts%2Fstreaming_merge.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZGF0YWZ1c2lvbi9waHlzaWNhbC1wbGFuL3NyYy9zb3J0cy9zdHJlYW1pbmdfbWVyZ2UucnM=) | 91.91% | [3 Missing and 5 partials :warning: ](https://app.codecov.io/gh/apache/datafusion/pull/27243?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/spill.rs](https://app.codecov.io/gh/apache/datafusion/pull/27243?src=pr&el=tree&filepath=datafusion%2Fphysical-plan%2Fsrc%2Faggregates%2Fspill.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZGF0YWZ1c2lvbi9waHlzaWNhbC1wbGFuL3NyYy9hZ2dyZWdhdGVzL3NwaWxsLnJz) | 81.81% | [0 Missing and 2 partials :warning: ](https://app.codecov.io/gh/apache/datafusion/pull/27243?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | | [Flag](https://app.codecov.io/gh/apache/datafusion/pull/27243/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | | |---|---|---| | [all-tests](https://app.codecov.io/gh/apache/datafusion/pull/27243/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `83.28% <91.48%> (+0.01%)` | :arrow_up: | | [sqllogictests](https://app.codecov.io/gh/apache/datafusion/pull/27243/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `60.82% <86.42%> (+0.10%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#carryforward-flags-in-the-pull-request-comment) to find out more. ```diff @@ Coverage Diff @@ ## branch-56 #27243 +/- ## ============================================= + Coverage 83.27% 83.28% +0.01% ============================================= Files 1169 1170 +1 Lines 474577 475237 +660 Branches 474577 475237 +660 ============================================= + Hits 395194 395804 +610 - Misses 55458 55481 +23 - Partials 23925 23952 +27 ``` | [Files with missing lines](https://app.codecov.io/gh/apache/datafusion/pull/27243?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | | |---|---|---| | [...ical-plan/src/aggregates/clustered\_final\_stream.rs](https://app.codecov.io/gh/apache/datafusion/pull/27243?src=pr&el=tree&filepath=datafusion%2Fphysical-plan%2Fsrc%2Faggregates%2Fclustered_final_stream.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZGF0YWZ1c2lvbi9waHlzaWNhbC1wbGFuL3NyYy9hZ2dyZWdhdGVzL2NsdXN0ZXJlZF9maW5hbF9zdHJlYW0ucnM=) | `88.91% <ø> (ø)` | | | [datafusion/physical-plan/src/sorts/builder.rs](https://app.codecov.io/gh/apache/datafusion/pull/27243?src=pr&el=tree&filepath=datafusion%2Fphysical-plan%2Fsrc%2Fsorts%2Fbuilder.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZGF0YWZ1c2lvbi9waHlzaWNhbC1wbGFuL3NyYy9zb3J0cy9idWlsZGVyLnJz) | `98.69% <100.00%> (ø)` | | | [datafusion/physical-plan/src/sorts/merge.rs](https://app.codecov.io/gh/apache/datafusion/pull/27243?src=pr&el=tree&filepath=datafusion%2Fphysical-plan%2Fsrc%2Fsorts%2Fmerge.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZGF0YWZ1c2lvbi9waHlzaWNhbC1wbGFuL3NyYy9zb3J0cy9tZXJnZS5ycw==) | `92.85% <100.00%> (+0.39%)` | :arrow_up: | | [...usion/physical-plan/src/sorts/multi\_level\_merge.rs](https://app.codecov.io/gh/apache/datafusion/pull/27243?src=pr&el=tree&filepath=datafusion%2Fphysical-plan%2Fsrc%2Fsorts%2Fmulti_level_merge.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZGF0YWZ1c2lvbi9waHlzaWNhbC1wbGFuL3NyYy9zb3J0cy9tdWx0aV9sZXZlbF9tZXJnZS5ycw==) | `89.67% <100.00%> (+0.06%)` | :arrow_up: | | [datafusion/physical-plan/src/aggregates/spill.rs](https://app.codecov.io/gh/apache/datafusion/pull/27243?src=pr&el=tree&filepath=datafusion%2Fphysical-plan%2Fsrc%2Faggregates%2Fspill.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZGF0YWZ1c2lvbi9waHlzaWNhbC1wbGFuL3NyYy9hZ2dyZWdhdGVzL3NwaWxsLnJz) | `91.39% <81.81%> (-0.81%)` | :arrow_down: | | [...afusion/physical-plan/src/sorts/streaming\_merge.rs](https://app.codecov.io/gh/apache/datafusion/pull/27243?src=pr&el=tree&filepath=datafusion%2Fphysical-plan%2Fsrc%2Fsorts%2Fstreaming_merge.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZGF0YWZ1c2lvbi9waHlzaWNhbC1wbGFuL3NyYy9zb3J0cy9zdHJlYW1pbmdfbWVyZ2UucnM=) | `93.29% <91.91%> (+0.52%)` | :arrow_up: | | [datafusion/physical-plan/src/sorts/stream.rs](https://app.codecov.io/gh/apache/datafusion/pull/27243?src=pr&el=tree&filepath=datafusion%2Fphysical-plan%2Fsrc%2Fsorts%2Fstream.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZGF0YWZ1c2lvbi9waHlzaWNhbC1wbGFuL3NyYy9zb3J0cy9zdHJlYW0ucnM=) | `85.43% <90.09%> (+1.00%)` | :arrow_up: | | [datafusion/physical-plan/src/spill/mod.rs](https://app.codecov.io/gh/apache/datafusion/pull/27243?src=pr&el=tree&filepath=datafusion%2Fphysical-plan%2Fsrc%2Fspill%2Fmod.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZGF0YWZ1c2lvbi9waHlzaWNhbC1wbGFuL3NyYy9zcGlsbC9tb2QucnM=) | `87.79% <87.61%> (+0.17%)` | :arrow_up: | | [...usion/physical-plan/src/spill/spilled\_row\_sizes.rs](https://app.codecov.io/gh/apache/datafusion/pull/27243?src=pr&el=tree&filepath=datafusion%2Fphysical-plan%2Fsrc%2Fspill%2Fspilled_row_sizes.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-ZGF0YWZ1c2lvbi9waHlzaWNhbC1wbGFuL3NyYy9zcGlsbC9zcGlsbGVkX3Jvd19zaXplcy5ycw==) | `92.72% <92.72%> (ø)` | | ... and [8 files with indirect coverage changes](https://app.codecov.io/gh/apache/datafusion/pull/27243/indirect-changes?src=pr&el=tree-more&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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
