[improve][broker] check if the owner is active when returning ownership (ExtensibleLoadManager)

[heesung-sn]

Motivation

When a broker shutdown or get killed, the leader broker will get notified and soon reassign the orphan bundle ownerships. Meanwhile, the ownwershiop lookup to these orphan bundles should be deferred til the new brokers own them. This will further reduce the lookup request retries when brokers are shutdown or get killed.

Modifications

In ServiceUnitStateChannelImpl.getOwnerAsync, additionally, check if the current owner broker is active or not in the broker registry. If active, return the current owner. Otherwise, defer the ownership lookup requests til the new owner owns it.

Verifying this change

• Make sure that the change passes the CI checks.

Added a test case testActiveGetOwner to cover this.

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: