pinot icon indicating copy to clipboard operation
pinot copied to clipboard

Migrate tests and connector to PinotAdminClient

Open xiangfu0 opened this issue 4 months ago • 1 comments

add missing admin clients (logical table, cluster, query workload); replace ControllerRequestClient usages where possible; add pinot-java-client dep in Flink connector

xiangfu0 avatar Nov 08 '25 01:11 xiangfu0

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 55.53%. Comparing base (8de3f2f) to head (e33bbd6). :warning: Report is 1 commits behind head on master.

:exclamation: There is a different number of reports uploaded between BASE (8de3f2f) and HEAD (e33bbd6). Click for more details.

HEAD has 32 uploads less than BASE
Flag BASE (8de3f2f) HEAD (e33bbd6)
java-21 5 1
unittests 4 2
temurin 10 2
java-11 5 1
unittests2 2 0
integration 6 0
integration2 2 0
integration1 2 0
custom-integration1 2 0
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #17167      +/-   ##
============================================
- Coverage     63.18%   55.53%   -7.66%     
+ Complexity     1479      703     -776     
============================================
  Files          3173     2474     -699     
  Lines        189917   140072   -49845     
  Branches      29064    22333    -6731     
============================================
- Hits         119994    77782   -42212     
+ Misses        60596    55731    -4865     
+ Partials       9327     6559    -2768     
Flag Coverage Δ
custom-integration1 ?
integration ?
integration1 ?
integration2 ?
java-11 55.50% <ø> (-7.63%) :arrow_down:
java-21 55.48% <ø> (-7.66%) :arrow_down:
temurin 55.53% <ø> (-7.66%) :arrow_down:
unittests 55.53% <ø> (-7.65%) :arrow_down:
unittests1 55.53% <ø> (-0.01%) :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.

:rocket: New features to boost your workflow:
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov-commenter avatar Nov 08 '25 02:11 codecov-commenter