celeborn icon indicating copy to clipboard operation
celeborn copied to clipboard

Published 20 hours ago • verified publisher icon apache

[CELEBORN-1334][BUILD] Client side should not import unnecessary deps

Open AngersZhuuuu opened this issue 11 months ago • 3 comments

What changes were proposed in this pull request?

Current spark client pom import hadoop client jar and jdk tool jar image

Client side should not contain such pom deps

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

AngersZhuuuu avatar Mar 15 '24 08:03 AngersZhuuuu

Codecov Report

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

Project coverage is 48.79%. Comparing base (96df0d6) to head (723d2a0). Report is 16 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #2394      +/-   ##
==========================================
+ Coverage   48.74%   48.79%   +0.05%     
==========================================
  Files         209      209              
  Lines       13021    13109      +88     
  Branches     1121     1134      +13     
==========================================
+ Hits         6346     6395      +49     
- Misses       6264     6292      +28     
- Partials      411      422      +11

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

codecov[bot] avatar Mar 15 '24 08:03 codecov[bot]

This PR is stale because it has been open 20 days with no activity. Remove stale label or comment or this will be closed in 10 days.

github-actions[bot] avatar Apr 17 '24 08:04 github-actions[bot]

Codecov Report

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

Project coverage is 48.79%. Comparing base (96df0d6) to head (723d2a0). Report is 69 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #2394      +/-   ##
==========================================
+ Coverage   48.74%   48.79%   +0.05%     
==========================================
  Files         209      209              
  Lines       13021    13109      +88     
  Branches     1121     1134      +13     
==========================================
+ Hits         6346     6395      +49     
- Misses       6264     6292      +28     
- Partials      411      422      +11

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

codecov-commenter avatar Apr 17 '24 08:04 codecov-commenter