Move version info outside of HB

[heroicNeZha]

What type of PR is this?

• bug
• feature
• enhancement

What does this PR do?

1. Move the version info outside the heartbeat
2. Save the version info when establishing the connection to metad server.

Which issue(s)/PR(s) this PR relates to?

#3055

Special notes for your reviewer, ex. impact of this fix, etc:

Additional context:

Checklist：

• Documentation affected （Please add the label if documentation needs to be modified.)
• Incompatible （If it is incompatible, please describe it and add corresponding label.）
• Need to cherry-pick （If need to cherry-pick to some branches, please label the destination version(s).)
• Performance impacted: Consumes more CPU/Memory

Release notes：

Please confirm whether to reflect in release notes and how to describe:

                                                            `

[darionyaphet]

why remove the version from heartbeat? to fix any problem?

[heroicNeZha]

why remove the version from heartbeat? to fix any problem?

it seems not necessary

[codecov-commenter]

Codecov Report

Merging #3378 (5e5b2a5) into master (fe68b0e) will decrease coverage by 0.00%.
The diff coverage is 96.25%.

@@            Coverage Diff             @@
##           master    #3378      +/-   ##
==========================================
- Coverage   85.27%   85.26%   -0.01%     
==========================================
  Files        1276     1277       +1     
  Lines      118999   119064      +65     
==========================================
+ Hits       101475   101520      +45     
- Misses      17524    17544      +20     

Impacted Files	Coverage Δ
src/meta/processors/admin/HBProcessor.cpp	92.30% <ø> (-0.38%)	⬇️
src/common/expression/ContainerExpression.cpp	90.00% <66.66%> (-2.13%)	⬇️
src/clients/meta/MetaClient.cpp	76.28% <100.00%> (+0.24%)	⬆️
src/common/utils/MetaKeyUtils.cpp	81.67% <100.00%> (+0.35%)	⬆️
src/graph/validator/test/YieldValidatorTest.cpp	91.78% <100.00%> (ø)
.../processors/admin/VerifyClientVersionProcessor.cpp	100.00% <100.00%> (ø)
src/meta/processors/parts/ListHostsProcessor.cpp	72.54% <100.00%> (+0.40%)	⬆️
src/meta/test/VerifyClientVersionTest.cpp	100.00% <100.00%> (ø)
src/parser/test/ParserTest.cpp	100.00% <100.00%> (ø)
src/meta/processors/parts/GetSpaceProcessor.cpp	60.60% <0.00%> (-18.19%)	⬇️
... and 23 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 71c73e1...5e5b2a5. Read the comment docs.

[darionyaphet]

Some people remember why there is a version number in their heartbeat? @yixinglu @Aiee @liuyu85cn

[yixinglu]

Some people remember why there is a version number in their heartbeat? @yixinglu @Aiee @liuyu85cn

Introduced in this commit

[HarrisChu]

How to keep the same version between storaged and metad?
e.g.
v2.5.1 storaged sends heartbeat to v2.6.1 metad

[heroicNeZha]

How to keep the same version between storaged and metad? e.g. v2.5.1 storaged sends heartbeat to v2.6.1 metad

by whitelist in metad: if v2.5.1 and v2.6.1 in whitelist, then they can in same cluster

[darionyaphet]

Cluster Manager is in progress.