[improve] [pip] PIP-357: Correct the conf name in load balance module.

[thetumbled]

Implementation PR: #22824

Motivation

We use loadBalancerBandwithInResourceWeight and loadBalancerBandwithOutResourceWeight to calculate the broker's load in the load balance module. However, the correct conf name should be loadBalancerBandwidthInResourceWeight and loadBalancerBandwidthOutResourceWeight. This PIP is to correct the conf name in the load balance module.

Modifications

Rename loadBalancerBandwithInResourceWeight to loadBalancerBandwidthInResourceWeight and loadBalancerBandwithOutResourceWeight to loadBalancerBandwidthOutResourceWeight in the load balance module.

Verifying this change

• Make sure that the change passes the CI checks.

(Please pick either of the following options)

This change is a trivial rework / code cleanup without any test coverage.

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

• Dependencies (add or upgrade a dependency)
• The public API
• The schema
• The default values of configurations
• The threading model
• The binary protocol
• The REST endpoints
• The admin CLI options
• The metrics
• Anything that affects deployment

Documentation

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

Matching PR in forked repository

PR in forked repository:

[heesung-sn]

This is very unfortunate.

Can we additionally parse those old ones in the Pulsar code and apply them instead of the new configs, if the new configs are 1.0 (default) but the old ones are not 1.0(non default) -- this means most likely the user still relies on the old one?

[Demogorgon314]

We can apply the following strategy to address the compatibility issue:

• If which configuration is not the default, then that one is used.
• If both the new and the old are configured, use the new one.

[thetumbled]

We can apply the following strategy to address the compatibility issue:

• If which configuration is not the default, then that one is used.
• If both the new and the old are configured, use the new one.

Agree, i have updated the doc, PTAL, thanks. @heesung-sn @Demogorgon314 @BewareMyPower @codelipenghui

[heesung-sn]

LGTM

[Technoboy-]

The vote passed. merging.