[fix][broker] Revert "[fix][load-balancer] skip mis-configured resource usage(>100%) in load balancer

[Technoboy-]

Motivation

Discuss is #18598

Documentation

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

[codelipenghui]

/pulsarbot run-failure-checks

[mattisonchao]

Error:  COMPILATION ERROR : 
[INFO] -------------------------------------------------------------
Error:  /home/runner/work/pulsar/pulsar/pulsar-common/src/test/java/org/apache/pulsar/policies/data/loadbalancer/LocalBrokerDataTest.java:[61,21] cannot find symbol
  symbol:   method getMaxResourceUsageWithWeightWithinLimit(double,double,double,double,double)
  location: variable data of type org.apache.pulsar.policies.data.loadbalancer.LocalBrokerData

Please take a look at this problem.

[codecov-commenter]

Codecov Report

Merging #18645 (55203aa) into master (b579bb8) will decrease coverage by 0.46%.
The diff coverage is 36.36%.

@@             Coverage Diff              @@
##             master   #18645      +/-   ##
============================================
- Coverage     47.51%   47.05%   -0.47%     
+ Complexity    10520    10416     -104     
============================================
  Files           698      698              
  Lines         68079    68056      -23     
  Branches       7280     7275       -5     
============================================
- Hits          32351    32021     -330     
- Misses        32151    32444     +293     
- Partials       3577     3591      +14     

Flag	Coverage Δ
unittests	47.05% <36.36%> (-0.47%)	⬇️

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

Impacted Files	Coverage Δ
...loadbalance/impl/LeastResourceUsageWithWeight.java	0.00% <0.00%> (ø)
...lsar/broker/loadbalance/impl/ThresholdShedder.java	30.32% <28.57%> (-17.39%)	⬇️
...roker/loadbalance/impl/ModularLoadManagerImpl.java	63.42% <66.66%> (-5.81%)	⬇️
...ar/broker/loadbalance/impl/BundleSplitterTask.java	60.00% <0.00%> (-17.78%)	⬇️
.../pulsar/broker/service/SharedConsumerAssignor.java	61.11% <0.00%> (-16.67%)	⬇️
.../apache/pulsar/broker/admin/impl/PackagesBase.java	54.12% <0.00%> (-13.77%)	⬇️
...e/pulsar/broker/service/EntryBatchIndexesAcks.java	82.14% <0.00%> (-10.72%)	⬇️
...er/loadbalance/impl/ModularLoadManagerWrapper.java	78.04% <0.00%> (-7.32%)	⬇️
...ava/org/apache/pulsar/broker/service/Consumer.java	64.59% <0.00%> (-7.30%)	⬇️
... and 42 more

[Technoboy-]

Error:  COMPILATION ERROR : 
[INFO] -------------------------------------------------------------
Error:  /home/runner/work/pulsar/pulsar/pulsar-common/src/test/java/org/apache/pulsar/policies/data/loadbalancer/LocalBrokerDataTest.java:[61,21] cannot find symbol
  symbol:   method getMaxResourceUsageWithWeightWithinLimit(double,double,double,double,double)
  location: variable data of type org.apache.pulsar.policies.data.loadbalancer.LocalBrokerData

Please take a look at this problem.

Fixed.

[Technoboy-]

@heesung-sn Could you please help review this ?

[hangc0276]

LGTM