[ISSUE #8970] Remove redundant heartbeats

[weihubeats]

#8970

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 9.09091% with 10 lines in your changes missing coverage. Please review.

Project coverage is 47.68%. Comparing base (9202de3) to head (39ef20b).
Report is 3 commits behind head on develop.

Files with missing lines	Patch %	Lines
...rocketmq/client/impl/factory/MQClientInstance.java	9.09%	10 Missing ⚠️

Additional details and impacted files

@@              Coverage Diff              @@
##             develop    #8971      +/-   ##
=============================================
- Coverage      47.73%   47.68%   -0.06%     
+ Complexity     11774    11765       -9     
=============================================
  Files           1304     1304              
  Lines          91118    91163      +45     
  Branches       11692    11703      +11     
=============================================
- Hits           43495    43469      -26     
- Misses         42264    42313      +49     
- Partials        5359     5381      +22     

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

---

🚨 Try these New Features:

• Flaky Tests Detection - Detect and resolve failed and flaky tests

[RongtongJin]

LGTM

[qianye1001]

it is used to make sure any proxy can receive heartbeat request before receive send or pull request when some proxy services share LB

[lizhimins]

Please look the previous comment

[weihubeats]

it is used to make sure any proxy can receive heartbeat request before receive send or pull request when some proxy services share LB

Can you give me an example?

[qianye1001]

it is used to make sure any proxy can receive heartbeat request before receive send or pull request when some proxy services share LB

Can you give me an example?

you can see how GO_AWAY works in remoting protocol to make proxy shutdown gracefully
#8599

when client receive GO_AWAY from proxy, client will reconnect proxy and send heartbeat first.

[weihubeats]

We can track connections to the broker where the initial connection triggers client-initiated heartbeats, while reconnections due to go away will have their heartbeats triggered by the onChannelActive even

[weihubeats]

@qianye1001 How does this sit with you

[RongtongJin]

LGTM

Sorry, I didn't realize it would affect the lossless release code