Closed ZuLiangWang closed 1 year ago
Merging #172 (8f0fc6f) into main (7f1354e) will decrease coverage by
0.58%
. The diff coverage is20.54%
.
@@ Coverage Diff @@
## main #172 +/- ##
==========================================
- Coverage 39.66% 39.08% -0.58%
==========================================
Files 34 34
Lines 4213 4306 +93
==========================================
+ Hits 1671 1683 +12
- Misses 2317 2398 +81
Partials 225 225
Impacted Files | Coverage Δ | |
---|---|---|
server/cluster/metadata/cluster_metadata.go | 0.00% <0.00%> (ø) |
|
server/cluster/metadata/table_manager.go | 0.00% <0.00%> (ø) |
|
server/cluster/metadata/topology_manager.go | 0.00% <0.00%> (ø) |
|
server/cluster/metadata/types.go | 0.00% <0.00%> (ø) |
|
server/coordinator/factory.go | 4.45% <0.00%> (ø) |
|
server/coordinator/procedure/manager_impl.go | 0.00% <0.00%> (ø) |
|
server/storage/meta.go | 0.00% <ø> (ø) |
|
server/storage/storage_impl.go | 57.21% <0.00%> (-5.10%) |
:arrow_down: |
server/id/id_impl.go | 63.73% <75.00%> (ø) |
|
server/storage/types.go | 63.34% <77.41%> (-0.20%) |
:arrow_down: |
... and 2 more |
Which issue does this PR close?
Closes #
Rationale for this change
In order to avoid having to build a new CeresMeta cluster every time a new CeresDB cluster is created, we need to support the feature of one CeresMeta cluster managing multiple CeresDB clusters.
What changes are included in this PR?
Are there any user-facing changes?
None.
How does this change test
Pass all unit tests and local manual tests.