radondb / radon

RadonDB is an open source, cloud-native MySQL database for building global, scalable cloud services
https://radondb.io/
GNU General Public License v3.0
1.79k stars 218 forks source link

*: fix the backendnum statistics error #698 #699

Closed zhyass closed 3 years ago

zhyass commented 3 years ago

[summary] The slave node of Radon will synchronize metadata from the master node. At this time, backend.json will be reloaded, causing backendnum to be counted repeatedly.

Set 0 before reload the backend config.

[test case] src/monitor/monitor_test.go

[patch codecov] src/monitor/monitor.go 96.3% src/backend/scatter.go 88.7%

codecov[bot] commented 3 years ago

Codecov Report

Merging #699 into master will decrease coverage by 0%. The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #699   +/-   ##
=====================================
- Coverage      89%    89%   -1%     
=====================================
  Files         130    130           
  Lines        9279   9281    +2     
=====================================
- Hits         8269   8267    -2     
- Misses        669    671    +2     
- Partials      341    343    +2     
Impacted Files Coverage Δ
src/backend/scatter.go 81% <100%> (+<1%) :arrow_up:
src/monitor/monitor.go 93% <100%> (+<1%) :arrow_up:
src/executor/engine/merge_join.go 95% <0%> (-2%) :arrow_down:
src/backend/txn.go 87% <0%> (-1%) :arrow_down:

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 af586a4...edd62cc. Read the comment docs.