Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[fix] [conf] fix configuration name and typo. #22822

Merged
merged 1 commit into from
Jun 4, 2024

Conversation

thetumbled
Copy link
Member

@thetumbled thetumbled commented Jun 3, 2024

Motivation

Some conf name is spell incorrectly in conf file deployment/terraform-ansible/templates/broker.conf, and there are many typo errors.

Modifications

  • correct the conf name: replicatioPolicyCheckDurationSeconds to replicationPolicyCheckDurationSeconds.
  • fix the typo errors.

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:

@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Jun 3, 2024
@BewareMyPower BewareMyPower merged commit 9454985 into apache:master Jun 4, 2024
24 of 26 checks passed
hanmz pushed a commit to hanmz/pulsar that referenced this pull request Feb 12, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants