Sitelet https://web.archive.org/web/20220323204757/https://github.com/apache/pulsar/pull/7926
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.

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

fix publish buffer of one ServerCnx caculated multi-times when check broker's publish buffer #7926

Merged
merged 1 commit into from Sep 8, 2020

Conversation

aloyszhang
Copy link
Contributor

@aloyszhang aloyszhang commented Aug 28, 2020 •

Fixes #7925

Master Issue: #

Motivation

This PR fix publish buffer of one ServerCnx caculated multi-times when check broker's publish buffer.

Modifications

check publish buffer of broker by adding messagePublishBufferSize of a ServerCnx for only one time.

@codelipenghui
Copy link
Contributor

@codelipenghui codelipenghui commented Aug 30, 2020

/pulsarbot run-failure-checks

@aloyszhang
Copy link
Contributor Author

@aloyszhang aloyszhang commented Sep 7, 2020

/pulsarbot run-failure-checks

jiazhai
jiazhai approved these changes Sep 8, 2020
@codelipenghui codelipenghui merged commit c9ede6e into apache:master Sep 8, 2020
20 checks passed
lbenc135 pushed a commit to lbenc135/pulsar that referenced this issue Oct 3, 2020
…broker publish buffer (apache#7926)

Fixes apache#7925 

Master Issue: #<xyz>

### Motivation

 This PR fix publish buffer of one ServerCnx caculated multi-times when check broker's publish buffer.

### Modifications

 check publish buffer of broker by adding messagePublishBufferSize of a ServerCnx for only one time.
wolfstudy added a commit that referenced this issue Oct 30, 2020
…broker publish buffer (#7926)

Fixes #7925 

Master Issue: #<xyz>

### Motivation

 This PR fix publish buffer of one ServerCnx caculated multi-times when check broker's publish buffer.

### Modifications

 check publish buffer of broker by adding messagePublishBufferSize of a ServerCnx for only one time.

(cherry picked from commit c9ede6e)
@aloyszhang aloyszhang deleted the publish-buffer branch Dec 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

4 participants