pinot icon indicating copy to clipboard operation
pinot copied to clipboard

Replace the usage of commons-collections to commons-collections4

Open xiangfu0 opened this issue 1 year ago • 1 comments

Instructions:

  1. The PR has to be tagged with at least one of the following labels (*):
    1. feature
    2. bugfix
    3. performance
    4. ui
    5. backward-incompat
    6. release-notes (**)
  2. Remove these instructions before publishing the PR.

(*) Other labels to consider:

  • testing
  • dependencies
  • docker
  • kubernetes
  • observability
  • security
  • code-style
  • extension-point
  • refactor
  • cleanup

(**) Use release-notes label for scenarios like:

  • New configuration options
  • Deprecation of configurations
  • Signature changes to public methods/interfaces
  • New plugins added or old plugins removed

xiangfu0 avatar Mar 23 '24 01:03 xiangfu0

Codecov Report

Attention: Patch coverage is 0% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 34.55%. Comparing base (59551e4) to head (23539cc). Report is 161 commits behind head on master.

Files Patch % Lines
...pache/pinot/core/util/PeerServerSegmentFinder.java 0.00% 1 Missing :warning:
Additional details and impacted files
@@              Coverage Diff              @@
##             master   #12711       +/-   ##
=============================================
- Coverage     61.75%   34.55%   -27.21%     
+ Complexity      207        6      -201     
=============================================
  Files          2436     2381       -55     
  Lines        133233   130758     -2475     
  Branches      20636    20253      -383     
=============================================
- Hits          82274    45177    -37097     
- Misses        44911    82260    +37349     
+ Partials       6048     3321     -2727     
Flag Coverage Δ
custom-integration1 <0.01% <0.00%> (-0.01%) :arrow_down:
integration <0.01% <0.00%> (-0.01%) :arrow_down:
integration1 <0.01% <0.00%> (-0.01%) :arrow_down:
integration2 0.00% <0.00%> (ø)
java-11 34.49% <0.00%> (-27.22%) :arrow_down:
java-21 34.43% <0.00%> (-27.20%) :arrow_down:
skip-bytebuffers-false 34.52% <0.00%> (-27.23%) :arrow_down:
skip-bytebuffers-true 34.41% <0.00%> (+6.68%) :arrow_up:
temurin 34.55% <0.00%> (-27.21%) :arrow_down:
unittests 46.16% <0.00%> (-15.59%) :arrow_down:
unittests1 46.16% <0.00%> (-0.73%) :arrow_down:
unittests2 ?

Flags with carried forward coverage won't be shown. Click here to find out more.

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

codecov-commenter avatar Mar 23 '24 02:03 codecov-commenter