Skip to content

Conversation

@dongzhonghua
Copy link
Contributor

@dongzhonghua dongzhonghua commented Oct 22, 2025

Motivation

add overrideBrokerNics for adaptation to heterogeneous network environments

Modifications

  • add loadBalancerOverrideBrokerNicSpeedGbps configuration, use overrideBrokerNics if loadBalancerOverrideBrokerNicSpeedGbps is not empty else get usable physical NICs.

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Matching PR in forked repository

PR in forked repository:

@github-actions
Copy link

@dongzhonghua Please add the following content to your PR description and select a checkbox:

- [ ] `doc` <!-- Your PR contains doc changes -->
- [ ] `doc-required` <!-- Your PR changes impact docs and you will update later -->
- [ ] `doc-not-needed` <!-- Your PR changes do not impact docs -->
- [ ] `doc-complete` <!-- Docs have been already added -->

@github-actions github-actions bot added doc-not-needed Your PR changes do not impact docs and removed doc-label-missing labels Oct 22, 2025
@Technoboy- Technoboy- added this to the 4.2.0 milestone Oct 23, 2025
dongzhonghua03 added 2 commits October 23, 2025 17:39
@codecov-commenter
Copy link

codecov-commenter commented Oct 26, 2025

Codecov Report

❌ Patch coverage is 80.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 74.28%. Comparing base (7c343d0) to head (695852e).
⚠️ Report is 49 commits behind head on master.

Files with missing lines Patch % Lines
.../apache/pulsar/broker/tools/LoadReportCommand.java 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##             master   #24883      +/-   ##
============================================
+ Coverage     74.24%   74.28%   +0.04%     
- Complexity    33911    34069     +158     
============================================
  Files          1913     1920       +7     
  Lines        149510   150305     +795     
  Branches      17373    17450      +77     
============================================
+ Hits         110997   111648     +651     
- Misses        29685    29754      +69     
- Partials       8828     8903      +75     
Flag Coverage Δ
inttests 26.44% <60.00%> (+0.15%) ⬆️
systests 22.78% <60.00%> (+0.06%) ⬆️
unittests 73.82% <80.00%> (+0.04%) ⬆️

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

Files with missing lines Coverage Δ
...org/apache/pulsar/broker/ServiceConfiguration.java 98.22% <100.00%> (+0.02%) ⬆️
...ker/loadbalance/impl/LinuxBrokerHostUsageImpl.java 86.36% <100.00%> (+6.67%) ⬆️
.../apache/pulsar/broker/tools/LoadReportCommand.java 6.06% <0.00%> (ø)

... and 114 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Technoboy- Technoboy- changed the title [improve] [broker] add overrideBrokerNics for adaptation of heterogeneous network environments [improve] [broker] Add overrideBrokerNics for adaptation of heterogeneous network environments Dec 2, 2025
@Technoboy- Technoboy- merged commit a272c5b into apache:master Dec 2, 2025
53 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/broker doc-not-needed Your PR changes do not impact docs ready-to-test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants