-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes not owned bundles got selected when broker overloading #5002
Fixes not owned bundles got selected when broker overloading #5002
Conversation
run java8 tests |
@kezhuw thank you for the fix. although just out of curiosity, at which condition, |
@sijie I found this issue in reading through I think the testing code is somewhat confusing, may be better to bind ownerless bundles to a not overloaded broker ? |
@sijie I have add a fixup commit to refine testing code, and now it should be more clearly that the testing code is trying to assert |
run cpp tests |
run java8 tests |
@merlimat @rdhabalia PTAL |
@merlimat @rdhabalia PTAL |
Motivation
Fixes not owned bundles got selected when broker overloading
Modifications
Filters out not owned bundles in finding candidate bundles for unloading.
Verifying this change
This change added tests and can be verified as follows:
Does this pull request potentially affect one of the following parts:
Documentation