[fix][broker] fix the non-active bundle clean logic when updating bundles

[aloyszhang]

Motivation

Pulsar saves all bundles load informations in loadData which contains bundleData,
the broker will update the bundleData by traversing all LocalBrokerData reported by every broker and remove non-active bundles (found in loadData but not in the newest reported LocalBrokerData)

the current removing of non-active happens in the wrong place, see

https://github.com/apache/pulsar/blob/master/pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/ModularLoadManagerImpl.java#L540-L571

it removes non-active bundle for every broker

Modifications

fix the non-active bundle clean logic when updating bundles.

remove non-active bundle after traverse brokers bundle data

Documentation

• doc-not-needed

Matching PR in forked repository

PR in forked repository: aloyszhang#18

[github-actions]

@aloyszhang 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 -->

[lifepuzzlefun]

@aloyszhang hi this pr duplicated with #20620 can you help review ? :-)

[aloyszhang]

@lifepuzzlefun sure

[aloyszhang]

close since duplicate with #20620