apache / incubator-horaedb-meta

Meta service of HoraeDB cluster.
https://horaedb.apache.org
Apache License 2.0
27 stars 15 forks source link

refactor: compatible with topology persistence #191

Closed ZuLiangWang closed 1 year ago

ZuLiangWang commented 1 year ago

Which issue does this PR close?

Closes #

Rationale for this change

We modified the persistent data TopologyType in the previous pull request. In order to avoid the user from manually operating the interface, we need to add compatible code

What changes are included in this PR?

Are there any user-facing changes?

Avoid users manually operating the interface to be compatible with version changes

How does this change test

An old version of the persistent data was used for testing in the local environment.

codecov[bot] commented 1 year ago

Codecov Report

Merging #191 (4b99389) into main (7f1354e) will increase coverage by 20.46%. The diff coverage is 55.36%.

@@             Coverage Diff             @@
##             main     #191       +/-   ##
===========================================
+ Coverage   39.66%   60.12%   +20.46%     
===========================================
  Files          34       41        +7     
  Lines        4213     4918      +705     
===========================================
+ Hits         1671     2957     +1286     
+ Misses       2317     1581      -736     
- Partials      225      380      +155     
Impacted Files Coverage Δ
pkg/coderr/code.go 0.00% <ø> (ø)
server/cluster/metadata/types.go 0.00% <0.00%> (ø)
server/coordinator/procedure/util.go 0.00% <ø> (ø)
server/storage/meta.go 0.00% <ø> (ø)
server/storage/storage_impl.go 57.21% <0.00%> (-5.10%) :arrow_down:
server/member/watch_leader.go 40.62% <20.00%> (-2.71%) :arrow_down:
server/cluster/manager.go 39.02% <23.07%> (ø)
server/member/member.go 47.39% <45.00%> (-1.32%) :arrow_down:
server/coordinator/scheduler/scheduler_manager.go 53.96% <47.36%> (ø)
server/coordinator/procedure/manager_impl.go 73.68% <55.55%> (+73.68%) :arrow_up:
... and 13 more

... and 3 files with indirect coverage changes