incubator-uniffle icon indicating copy to clipboard operation
incubator-uniffle copied to clipboard

[#1327]feat(tez): Support combine operation in sort for tez engine.

Open qijiale76 opened this issue 1 year ago • 1 comments

What changes were proposed in this pull request?

Support combine operation in sort for tez engine.

Why are the changes needed?

Fix: #1327

Does this PR introduce any user-facing change?

No.

How was this patch tested?

new UT

qijiale76 avatar Nov 23 '23 12:11 qijiale76

Codecov Report

Attention: 13 lines in your changes are missing coverage. Please review.

Comparison is base (51e02d0) 53.47% compared to head (6dd9778) 54.50%.

Files Patch % Lines
...library/common/sort/buffer/WriteBufferManager.java 57.89% 7 Missing and 1 partial :warning:
...untime/library/common/sort/buffer/WriteBuffer.java 91.66% 4 Missing and 1 partial :warning:
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1328      +/-   ##
============================================
+ Coverage     53.47%   54.50%   +1.03%     
- Complexity     2695     2701       +6     
============================================
  Files           411      392      -19     
  Lines         23644    21355    -2289     
  Branches       2006     2013       +7     
============================================
- Hits          12643    11640    -1003     
+ Misses        10225     9008    -1217     
+ Partials        776      707      -69     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar Nov 23 '23 12:11 codecov-commenter