Skip to content
This repository has been archived by the owner on Apr 1, 2024. It is now read-only.

PR-20822: [fix][broker] Avoid infinite bundle unloading #5839

Open
1 of 4 tasks
streamnativebot opened this issue Jul 19, 2023 · 0 comments
Open
1 of 4 tasks

PR-20822: [fix][broker] Avoid infinite bundle unloading #5839

streamnativebot opened this issue Jul 19, 2023 · 0 comments

Comments

@streamnativebot
Copy link

url: apache#20822

Motivation

The bundle ownership assignment logic doesn't know the previous unloaded broker when unloading happens. It might assign the bundle to the same broker. In this case, it might cause an infinite bundle unloading loop.

To resolve this issue, we can check the destination broker when doing the doLoadShedding stage. When the destination broker is the same as the current owner broker, we can skip this unload, and if it is different, we set the new owner in this stage.

Modifications

  • Transfer the bundle to the new owner when needed to unload.

Verifying this change

See the ModularLoadManagerImplTest#testLoadShedding unit test.

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant