Skip to content

[improve][broker] Add methods for batch updating and deleting bookie rack information#22857

Open
hrzzzz wants to merge 2 commits intoapache:masterfrom
hrzzzz:bookie-rack-bulk-ops
Open

[improve][broker] Add methods for batch updating and deleting bookie rack information#22857
hrzzzz wants to merge 2 commits intoapache:masterfrom
hrzzzz:bookie-rack-bulk-ops

Conversation

@hrzzzz
Copy link
Contributor

@hrzzzz hrzzzz commented Jun 6, 2024

Motivation

We can add admin APIs for batch updating or deleting bookie rack info to reduce write operations to ZooKeeper.

Modifications

Add admin APIs:

  • batchDeleteBookiesRackInfo
  • batchUpdateBookiesRackInfo

Verifying this change

  • Make sure that the change passes the CI checks.

Added tests:

  • org.apache.pulsar.broker.admin.BookiesApiTest#testUpdateBookieRackInfo
  • org.apache.pulsar.broker.admin.BookiesApiTest#testDeleteBookieRackInfo
  • org.apache.pulsar.broker.admin.BookiesApiTest#testClearAllBookiesRackInfo

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: hrzzzz#6

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant