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

[improve][broker] defer the ownership checks if the owner is inactive (ExtensibleLoadManager) #21855

Merged
merged 1 commit into from
Jan 5, 2024
Merged

[improve][broker] defer the ownership checks if the owner is inactive (ExtensibleLoadManager) #21855

merged 1 commit into from
Jan 5, 2024
+177 −42

Conversation

heesung-sn
Copy link
Contributor

Cherry-picked #21811

@heesung-sn heesung-sn self-assigned this Jan 4, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 4, 2024

@heesung-sn Please add the following content to your PR description and select a checkbox:

- [ ] `doc` <!-- Your PR contains doc changes -->
- [ ] `doc-required` <!-- Your PR changes impact docs and you will update later -->
- [ ] `doc-not-needed` <!-- Your PR changes do not impact docs -->
- [ ] `doc-complete` <!-- Docs have been already added -->

dragosvictor
dragosvictor approved these changes Jan 4, 2024
View reviewed changes
Copy link
Contributor

@dragosvictor dragosvictor left a comment

Choose a reason for hiding this comment

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

LGTM!

@codelipenghui codelipenghui added ready-to-test doc-not-needed Your PR changes do not impact docs and removed doc-label-missing labels Jan 4, 2024
@codelipenghui
Copy link
Contributor

/pulsarbot run-failure-checks

@github-actions github-actions bot added doc-label-missing and removed doc-not-needed Your PR changes do not impact docs labels Jan 4, 2024
@Technoboy- Technoboy- merged commit fe1d9ef into apache:branch-3.1 Jan 5, 2024
49 of 52 checks passed
heesung-sn added a commit to heesung-sn/pulsar that referenced this pull request Jan 5, 2024
@heesung-sn
[improve][broker] defer the ownership checks if the owner is inactive…
857792f 
… (ExtensibleLoadManager) (apache#21855)
heesung-sn added a commit to heesung-sn/pulsar that referenced this pull request Jan 5, 2024
@heesung-sn
[improve][broker] defer the ownership checks if the owner is inactive…
df753c9 
… (ExtensibleLoadManager) (apache#21855)
mukesh-ctds pushed a commit to datastax/pulsar that referenced this pull request Mar 1, 2024
@heesung-sn @mukesh-ctds
[improve][broker] defer the ownership checks if the owner is inactive…
dd57072 
… (ExtensibleLoadManager) (apache#21855)

(cherry picked from commit fe1d9ef)
mukesh-ctds pushed a commit to datastax/pulsar that referenced this pull request Mar 6, 2024
@heesung-sn @mukesh-ctds
[improve][broker] defer the ownership checks if the owner is inactive…
5ff71e6 
… (ExtensibleLoadManager) (apache#21855)

(cherry picked from commit fe1d9ef)
@heesung-sn heesung-sn deleted the branch-3.1-cherry-pick branch April 2, 2024 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@codelipenghui codelipenghui codelipenghui approved these changes

@dragosvictor dragosvictor dragosvictor approved these changes

@Demogorgon314 Demogorgon314 Awaiting requested review from Demogorgon314

Assignees

@heesung-sn heesung-sn

Labels
doc-label-missing ready-to-test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@heesung-sn @codelipenghui @dragosvictor @Technoboy-