[ISSUE #9687] Cleanup BrokerHeartbeatManager code

[majialoong]

Which Issue(s) This PR Fixes

Fixes #9687

Brief Description

Cleanup BrokerHeartbeatManager unnecessary code.

[majialoong]

Hello, @lizhimins @RongtongJin , If have chance, please take a look this RP, thanks !

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 48.14%. Comparing base (a4da5f5) to head (4aafde8).

Additional details and impacted files

@@              Coverage Diff              @@
##             develop    #9688      +/-   ##
=============================================
- Coverage      48.21%   48.14%   -0.08%     
+ Complexity     12166    12150      -16     
=============================================
  Files           1313     1313              
  Lines          93335    93335              
  Branches       11962    11962              
=============================================
- Hits           45005    44935      -70     
- Misses         42762    42816      +54     
- Partials        5568     5584      +16     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[majialoong]

@RongtongJin Could you please take another look at this PR? The failed CI is not related to the current patch.

[RongtongJin]

@RongtongJin Could you please take another look at this PR? The failed CI is not related to the current patch.

Plz merge lastest develop branch code to pass the CI.

[majialoong]

@RongtongJin Could you please take another look at this PR? The failed CI is not related to the current patch.

Plz merge lastest develop branch code to pass the CI.

Thanks for the reminder, I have merged the latest code.