Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix RegionAwareEnsemblePlacementPolicy update rack info problem. #3666

Merged
merged 3 commits into from
Dec 7, 2022
Merged

Fix RegionAwareEnsemblePlacementPolicy update rack info problem. #3666

merged 3 commits into from
Dec 7, 2022

Conversation

horizonzy
Copy link
Member

Motivation

Fixes #3665

@hangc0276 hangc0276 added this to the 4.16.0 milestone Nov 23, 2022
@horizonzy
Copy link
Member Author

rerun failure checks

@hangc0276
Copy link
Contributor

ping @eolivelli @dlg99 @zymap @merlimat Please help take a look, thanks.

eolivelli
eolivelli approved these changes Dec 7, 2022
View reviewed changes
Copy link
Contributor

@eolivelli eolivelli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@eolivelli eolivelli merged commit 4574ba0 into apache:master Dec 7, 2022
hangc0276 pushed a commit that referenced this pull request Dec 8, 2022
@horizonzy @hangc0276
Fix RegionAwareEnsemblePlacementPolicy update rack info problem. (#3666)
b7f9b83 
(cherry picked from commit 4574ba0)
nicoloboschi pushed a commit to datastax/bookkeeper that referenced this pull request Jan 11, 2023
@horizonzy @nicoloboschi
Fix RegionAwareEnsemblePlacementPolicy update rack info problem. (apa…
d7eb40b 
…che#3666)

(cherry picked from commit 4574ba0)
(cherry picked from commit b7f9b83)
yaalsn pushed a commit to yaalsn/bookkeeper that referenced this pull request Jan 30, 2023
@horizonzy @yaalsn
Fix RegionAwareEnsemblePlacementPolicy update rack info problem. (apa…
bd2e303 
…che#3666)
zymap pushed a commit that referenced this pull request Feb 16, 2023
@horizonzy @zymap
Fix RegionAwareEnsemblePlacementPolicy update rack info problem. (#3666)
42504da 
(cherry picked from commit 4574ba0)
Ghatage pushed a commit to sijie/bookkeeper that referenced this pull request Jul 12, 2024
@horizonzy
Fix RegionAwareEnsemblePlacementPolicy update rack info problem. (apa…
ac451f4 
…che#3666)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hangc0276 hangc0276 hangc0276 approved these changes

@eolivelli eolivelli eolivelli approved these changes

@dlg99 dlg99 Awaiting requested review from dlg99

@merlimat merlimat Awaiting requested review from merlimat

@zymap zymap Awaiting requested review from zymap

Assignees

@horizonzy horizonzy

Labels
cherry-picked/branch-4.14 cherry-picked/branch-4.15 release/4.14.6 release/4.15.4 type/bug
Projects
None yet
Milestone
4.16.0
Development

Successfully merging this pull request may close these issues.

RegionAwareEnsemblePlacementPolicy#onBookieRackChange didn't update perRegionPlacement.
4 participants
@horizonzy @hangc0276 @eolivelli @zymap