Skip to content

[Broker]Move normalize to DispatchRateImpl#14737

Merged
codelipenghui merged 1 commit intoapache:masterfrom
AnonHxy:opt_DispatchRateImpl_nomalize
Mar 18, 2022
Merged

[Broker]Move normalize to DispatchRateImpl#14737
codelipenghui merged 1 commit intoapache:masterfrom
AnonHxy:opt_DispatchRateImpl_nomalize

Conversation

@AnonHxy
Copy link
Contributor

@AnonHxy AnonHxy commented Mar 18, 2022 •
edited
Loading

Motivation

Move normalize method to DispatchRateImpl as a static method like PublishRate and SubscribeRate

Modifications

Move normalize method to DispatchRateImpl as a static method

Verifying this change

  • Make sure that the change passes the CI checks.

Does this pull request potentially affect one of the following parts:

  • Dependencies (does it add or upgrade a dependency): (no)
  • The public API: (no)
  • The schema: (no)
  • The default values of configurations: (no)
  • The wire protocol: (no)
  • The rest endpoints: (no)
  • The admin cli options: (no)
  • Anything that affects deployment: (no)

Documentation

  • no-need-doc

@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Mar 18, 2022
Technoboy-
Technoboy- approved these changes Mar 18, 2022
View reviewed changes
Copy link
Contributor

@Technoboy- Technoboy- 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 merged commit 88d2cc0 into apache:master Mar 18, 2022
@codelipenghui codelipenghui added this to the 2.11.0 milestone Mar 18, 2022
aparajita89 pushed a commit to aparajita89/pulsar that referenced this pull request Mar 21, 2022
@AnonHxy
Move normalize in DispatchRateImpl (apache#14737)
8c9ff10
Nicklee007 pushed a commit to Nicklee007/pulsar that referenced this pull request Apr 20, 2022
@AnonHxy
Move normalize in DispatchRateImpl (apache#14737)
b589296
nicoloboschi pushed a commit to datastax/pulsar that referenced this pull request Sep 16, 2022
@AnonHxy @nicoloboschi
Move normalize in DispatchRateImpl (apache#14737)
0e409f0 
(cherry picked from commit 88d2cc0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Technoboy- Technoboy- Technoboy- approved these changes

@codelipenghui codelipenghui codelipenghui approved these changes

Assignees

@AnonHxy AnonHxy

Labels

doc-not-needed Your PR changes do not impact docs

Projects

None yet

Milestone

2.11.0

Development

Successfully merging this pull request may close these issues.

3 participants

@AnonHxy @Technoboy- @codelipenghui