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

[#128] improvement: add a integration test case for Spark with MiniYarnCluster

Open lifeSo opened this issue 2 years ago • 2 comments

What changes were proposed in this pull request?

Add a integration test case for Spark with MiniYarnCluster

Why are the changes needed?

Fix: https://github.com/apache/incubator-uniffle/issues/138

Does this PR introduce any user-facing change?

No.

How was this patch tested?

It self is test case

lifeSo avatar Dec 24 '23 14:12 lifeSo

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (ecbf2e7) 53.22% compared to head (e8746ec) 54.14%. Report is 19 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1395      +/-   ##
============================================
+ Coverage     53.22%   54.14%   +0.91%     
- Complexity     2715     2719       +4     
============================================
  Files           418      399      -19     
  Lines         23932    21605    -2327     
  Branches       2043     2042       -1     
============================================
- Hits          12738    11698    -1040     
+ Misses        10408     9190    -1218     
+ Partials        786      717      -69     

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

codecov-commenter avatar Dec 24 '23 14:12 codecov-commenter

The PR title should obey the rule like that [#128] improvement(tests): Add a integration test case for Spark with MiniYarnCluster

And If this is on progress, you can mark this PR as draft that could be finished by this button, image

zuston avatar Dec 25 '23 02:12 zuston