pinot icon indicating copy to clipboard operation
pinot copied to clipboard

Set SSL context in ControllerContext/BrokerContext/ServerContext

Open xiangfu0 opened this issue 3 months ago • 1 comments

xiangfu0 avatar Dec 12 '25 11:12 xiangfu0

Codecov Report

:x: Patch coverage is 31.20301% with 183 lines in your changes missing coverage. Please review. :white_check_mark: Project coverage is 63.17%. Comparing base (92b6182) to head (db413f5). :warning: Report is 7 commits behind head on master.

Files with missing lines Patch % Lines
...org/apache/pinot/query/mailbox/MailboxService.java 19.56% 35 Missing and 2 partials :warning:
.../pinot/common/utils/ClientSSLContextGenerator.java 24.13% 22 Missing :warning:
...apache/pinot/controller/BaseControllerStarter.java 0.00% 18 Missing :warning:
...che/pinot/core/instance/context/ServerContext.java 16.66% 15 Missing :warning:
...pinot/core/instance/context/ControllerContext.java 0.00% 14 Missing :warning:
...che/pinot/core/instance/context/BrokerContext.java 33.33% 12 Missing :warning:
...e/pinot/broker/broker/helix/BaseBrokerStarter.java 0.00% 10 Missing :warning:
.../pinot/server/starter/helix/BaseServerStarter.java 0.00% 10 Missing :warning:
...apache/pinot/client/SslContextProviderFactory.java 75.67% 7 Missing and 2 partials :warning:
.../pinot/query/service/dispatch/QueryDispatcher.java 27.27% 7 Missing and 1 partial :warning:
... and 7 more
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #17358      +/-   ##
============================================
- Coverage     63.20%   63.17%   -0.03%     
- Complexity     1476     1477       +1     
============================================
  Files          3170     3172       +2     
  Lines        189523   189751     +228     
  Branches      28997    29036      +39     
============================================
+ Hits         119779   119877      +98     
- Misses        60451    60566     +115     
- Partials       9293     9308      +15     
Flag Coverage Δ
custom-integration1 100.00% <ø> (ø)
integration 100.00% <ø> (ø)
integration1 100.00% <ø> (ø)
integration2 0.00% <ø> (ø)
java-11 63.13% <31.20%> (-0.03%) :arrow_down:
java-21 63.12% <31.20%> (-0.06%) :arrow_down:
temurin 63.17% <31.20%> (-0.03%) :arrow_down:
unittests 63.17% <31.20%> (-0.03%) :arrow_down:
unittests1 55.50% <19.37%> (-0.05%) :arrow_down:
unittests2 34.04% <25.56%> (+0.01%) :arrow_up:

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 Dec 12 '25 11:12 codecov-commenter