[Issue 14013][broker] make getInternalStats method async

[HQebupt]

Master Issue: #14013

Motivation

See #14013

Verifying this change

Make sure that the change passes the CI checks.

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

If yes was chosen, please highlight the changes

• Dependencies (does it add or upgrade a dependency): (no)
• The public API: (no)
• The schema: (no)
• The default values of configurations: (no)
• The wire protocol: (no)
• The rest endpoints: (no)
• The admin cli options: (no)
• Anything that affects deployment: (no)

Documentation

Check the box below and label this PR (if you have committer privilege).

Need to update docs?

• no-need-doc

[mattisonchao]

LGTM +1.
but I have two suggestions:

Remove asyncResponse from the internal method, and unified exception and result handling are performed at the controller layer, which makes the internal method purer.
Have you checked to see if there is a test for the modified method ?

[Jason918]

LGTM
Please fix test failure.

[HQebupt]

/pulsarbot rerun-failure-checks

[github-actions]

The pr had no activity for 30 days, mark with Stale label.

[Jason918]

@HQebupt Please resolve the test failure.

[HQebupt]

/pulsarbot rerun-failure-checks

[HQebupt]

/pulsarbot rerun-failure-checks

[lhotari]

Please don't rerun failed checks. That won't make the tests pass since this PR contains outdated GitHub Actions workflow changes. You must rebase this PR, push new commits or merge master branch changes to it. Please check the dev mailing list for more information.

[HQebupt]

Please don't rerun failed checks. That won't make the tests pass since this PR contains outdated GitHub Actions workflow changes. You must rebase this PR, push new commits or merge master branch changes to it. Please check the dev mailing list for more information.

Got it. Thx Lari @lhotari

[github-actions]

The pr had no activity for 30 days, mark with Stale label.

[github-actions]

The pr had no activity for 30 days, mark with Stale label.