Skip to content

[fix] chore: remove unused preciseTopicPublishRateLimiterEnable#23

Open
dlg99 wants to merge 1 commit intomasterfrom
precise-config
Open

[fix] chore: remove unused preciseTopicPublishRateLimiterEnable#23
dlg99 wants to merge 1 commit intomasterfrom
precise-config

Conversation

@dlg99
Copy link
Owner

@dlg99 dlg99 commented May 5, 2025

Motivation

apache#21681 removed actual uses of preciseTopicPublishRateLimiterEnable config but left some plumbing (setting/reading config) in the code.

Modifications

Removing unused code

Verifying this change

  • Make sure that the change passes the CI checks.

(Please pick either of the following options)

This change is a trivial rework / code cleanup without any test coverage.

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

If the box was checked, please highlight the changes

  • Dependencies (add or upgrade a dependency)
  • The public API
  • The schema
  • The default values of configurations
  • The threading model
  • The binary protocol
  • The REST endpoints
  • The admin CLI options
  • The metrics
  • Anything that affects deployment

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Matching PR in forked repository

PR in forked repository: #23

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant