Skip to content

[improve][broker] Fast Return the method computeLocalQuota in ResourceQuotaCalculatorImpl#18817

Merged
Technoboy- merged 1 commit intoapache:masterfrom
Nicklee007:fast-return-in-computeLocalQuota
Dec 8, 2022
Merged

[improve][broker] Fast Return the method computeLocalQuota in ResourceQuotaCalculatorImpl#18817
Technoboy- merged 1 commit intoapache:masterfrom
Nicklee007:fast-return-in-computeLocalQuota

Conversation

@Nicklee007
Copy link
Contributor

@Nicklee007 Nicklee007 commented Dec 8, 2022

Master Issue: #18586

Motivation

Optimize the method computeLocalQuota in ResourceQuotaCalculatorImpl, fast return when confUsage < 0.

Modifications

  1. Fast return when confUsage < 0 in ResourceQuotaCalculatorImpl#computeLocalQuota.

Documentation

  • doc-not-needed

Matching PR in forked repository

PR in forked repository: Nicklee007#4

@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Dec 8, 2022
@congbobo184 congbobo184 added this to the 2.12.0 milestone Dec 8, 2022
@Nicklee007
Copy link
Contributor Author

/pulsarbot run-failure-checks

@Technoboy- Technoboy- merged commit 30b52a1 into apache:master Dec 8, 2022
Demogorgon314 pushed a commit to Demogorgon314/pulsar that referenced this pull request Dec 26, 2022
…eQuotaCalculatorImpl (apache#18817)

Co-authored-by: nicklixinyang <nicklixinyang@didiglobal.com>
Demogorgon314 pushed a commit to Demogorgon314/pulsar that referenced this pull request Dec 29, 2022
…eQuotaCalculatorImpl (apache#18817)

Co-authored-by: nicklixinyang <nicklixinyang@didiglobal.com>
lifepuzzlefun pushed a commit to lifepuzzlefun/pulsar that referenced this pull request Jan 10, 2023
…eQuotaCalculatorImpl (apache#18817)

Co-authored-by: nicklixinyang <nicklixinyang@didiglobal.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs ready-to-test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants