Skip to content

[fix][admin] Refactor namespace migration operation to async in rest api - #25478

Merged
Technoboy- merged 2 commits into
apache:masterfrom
oneby-wang:enable_migration_async_refactor
Apr 7, 2026
Merged

[fix][admin] Refactor namespace migration operation to async in rest api#25478
Technoboy- merged 2 commits into
apache:masterfrom
oneby-wang:enable_migration_async_refactor

Conversation

@oneby-wang

Copy link
Copy Markdown
Contributor

Motivation

It is not recommended to use sync operations in NIO threads, which would block other operations and degrade the overall performance of the system.

Modifications

Refactor namespace migration sync operation to async in rest api, and add tests.

Verifying this change

  • Make sure that the change passes the CI checks.

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: oneby-wang#30

@github-actions github-actions Bot added the doc-not-needed Your PR changes do not impact docs label Apr 6, 2026
@Technoboy-
Technoboy- merged commit f72a5a7 into apache:master Apr 7, 2026
82 of 84 checks passed
@Technoboy- Technoboy- added this to the 4.3.0 milestone Apr 7, 2026
coderzc pushed a commit to coderzc/pulsar that referenced this pull request Apr 15, 2026
lhotari pushed a commit that referenced this pull request Apr 15, 2026
lhotari added a commit that referenced this pull request Apr 15, 2026
lhotari pushed a commit that referenced this pull request Apr 15, 2026
lhotari pushed a commit that referenced this pull request Apr 21, 2026
srinath-ctds pushed a commit to datastax/pulsar that referenced this pull request Apr 23, 2026
…api (apache#25478)

(cherry picked from commit f72a5a7)
(cherry picked from commit 066a3f3)
srinath-ctds pushed a commit to datastax/pulsar that referenced this pull request Apr 23, 2026
…in rest api (apache#25478)"

This reverts commit 066a3f3.

(cherry picked from commit eefa9f0)
srinath-ctds pushed a commit to datastax/pulsar that referenced this pull request Apr 23, 2026
…api (apache#25478)

(cherry picked from commit f72a5a7)
(cherry picked from commit b960c28)
nodece pushed a commit to ascentstream/pulsar that referenced this pull request May 27, 2026
nodece pushed a commit to ascentstream/pulsar that referenced this pull request May 27, 2026
nodece pushed a commit to ascentstream/pulsar that referenced this pull request May 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants