Skip to content

Remove unused params in initializeRateLimiterIfNeeded#14909

Merged
RobertIndie merged 1 commit intoapache:masterfrom
AnonHxy:remove_unused_parma
Mar 29, 2022
Merged

Remove unused params in initializeRateLimiterIfNeeded#14909
RobertIndie merged 1 commit intoapache:masterfrom
AnonHxy:remove_unused_parma

Conversation

@AnonHxy
Copy link
Contributor

@AnonHxy AnonHxy commented Mar 28, 2022

Motivation

Remove unused params in initializeRateLimiterIfNeeded

Modifications

Remove unused params in org.apache.pulsar.broker.service.persistent.PersistentTopic#initializeRateLimiterIfNeeded

Verifying this change

  • Make sure that the change passes the CI checks.

Documentation

  • no-need-doc

@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Mar 28, 2022
@AnonHxy
Copy link
Contributor Author

AnonHxy commented Mar 28, 2022

/pulsarbot run-failure-checks

@Jason918
Copy link
Contributor

/pulsarbot run-failure-checks

@RobertIndie RobertIndie added type/cleanup Code or doc cleanups e.g. remove the outdated documentation or remove the code no longer in use area/broker labels Mar 29, 2022
@RobertIndie RobertIndie merged commit 3b13830 into apache:master Mar 29, 2022
Nicklee007 pushed a commit to Nicklee007/pulsar that referenced this pull request Apr 20, 2022
…ed (apache#14909)

### Motivation

Remove unused params in initializeRateLimiterIfNeeded

### Modifications

Remove unused params in `org.apache.pulsar.broker.service.persistent.PersistentTopic#initializeRateLimiterIfNeeded`
AnonHxy added a commit to AnonHxy/pulsar that referenced this pull request Jul 25, 2022
…ed (apache#14909)

Remove unused params in initializeRateLimiterIfNeeded

Remove unused params in `org.apache.pulsar.broker.service.persistent.PersistentTopic#initializeRateLimiterIfNeeded`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/broker doc-not-needed Your PR changes do not impact docs type/cleanup Code or doc cleanups e.g. remove the outdated documentation or remove the code no longer in use

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants