Skip to content

[pulsar-broker] support secondary bookie isolation group at namespace #4458

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
Merged
rdhabalia merged 3 commits into from
Jun 6, 2019
Merged

[pulsar-broker] support secondary bookie isolation group at namespace #4458

rdhabalia merged 3 commits into from
Jun 6, 2019

Conversation

@rdhabalia
Copy link
Contributor

@rdhabalia rdhabalia commented Jun 4, 2019

Motivation

with #4261, broker supports secondary bk-isolation group. Now, namespace level bookie-isolation can also have secondary isolation group to fallback if primary isolation group doesn't have enough bookies available.

Modification

  • rest api change to accept bookieAffinityGroup data and cli/admin-client can pass the expected data.
  • bk-client can receive both primary and secondary isolation groups to create ensemble for ledgers of the namespace topics.

@rdhabalia rdhabalia added this to the 2.4.0 milestone Jun 4, 2019
@rdhabalia rdhabalia self-assigned this Jun 4, 2019
@jiazhai
Copy link
Member

jiazhai commented Jun 4, 2019

run java8 tests

@rdhabalia rdhabalia mentioned this pull request Jun 4, 2019
Merged
@rdhabalia
Copy link
Contributor Author

rdhabalia commented Jun 5, 2019

rerun cpp tests

@rdhabalia
Copy link
Contributor Author

rdhabalia commented Jun 6, 2019

@jiazhai @merlimat

merlimat
merlimat approved these changes Jun 6, 2019
View reviewed changes
Copy link
Contributor

@merlimat merlimat left a comment

Choose a reason for hiding this comment

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

👍

@rdhabalia rdhabalia merged commit ed836e2 into apache:master Jun 6, 2019
@rdhabalia rdhabalia deleted the bk_iso_sec branch June 6, 2019 22:27
sijie pushed a commit that referenced this pull request Jun 14, 2019
@rdhabalia @sijie
[pulsar-broker] add cli/admin api to delete bookie-affinity group (#4471
6636c79 
)

### Motivation

This PR is on top of #4458. It adds support to delete bookie-affinity group using cli/admin api.
@Jennifer88huang-zz Jennifer88huang-zz mentioned this pull request Jan 13, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@merlimat merlimat merlimat approved these changes

@sijie sijie Awaiting requested review from sijie

@jiazhai jiazhai Awaiting requested review from jiazhai

@jai1 jai1 Awaiting requested review from jai1

@massakam massakam Awaiting requested review from massakam

Assignees

@rdhabalia rdhabalia

Labels

None yet

Projects

None yet

Milestone

2.4.0

Development

Successfully merging this pull request may close these issues.

3 participants

@rdhabalia @jiazhai @merlimat