search

sofastack / sofa-registry

SOFARegistry is a production-level, low-latency, high-availability service registry powered by Ant Financial.
https://www.sofastack.tech/sofa-registry/docs/Home
Apache License 2.0
653 stars 247 forks source link

session multi sub counter metric #294

Closed nocvalight closed 1 year ago

nocvalight commented 1 year ago

Motivation:

Explain the context, and why you're making that change. To make others understand what is the problem you're trying to solve.

Modification:

Result:

Fixes #291 .

If there is no issue then describe the changes introduced by this PR.

codecov[bot] commented 1 year ago

Codecov Report

Merging #294 (88254e8) into master (24e3f9f) will increase coverage by 0.18%. The diff coverage is 93.22%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #294      +/-   ##
============================================
+ Coverage     76.33%   76.51%   +0.17%     
  Complexity       48       48              
============================================
  Files           492      492              
  Lines         18189    18239      +50     
  Branches       1780     1783       +3     
============================================
+ Hits          13885    13956      +71     
+ Misses         3501     3479      -22     
- Partials        803      804       +1
Impacted Files Coverage Δ
...ulti/cluster/slot/MultiClusterSlotManagerImpl.java 80.82% <50.00%> (+3.52%) :arrow_up:
...lipay/sofa/registry/server/data/timer/Metrics.java 90.90% <88.88%> (-2.85%) :arrow_down:
...stry/server/data/change/DataChangeEventCenter.java 96.19% <100.00%> (+0.72%) :arrow_up:
...er/session/scheduler/timertask/CacheCountTask.java 100.00% <100.00%> (ø)
...ry/server/session/scheduler/timertask/Metrics.java 98.78% <100.00%> (+0.25%) :arrow_up:
...ofa/registry/server/session/store/DataIndexer.java 82.25% <0.00%> (-8.07%) :arrow_down:
...egistry/common/model/appmeta/InterfaceMapping.java 80.00% <0.00%> (-6.67%) :arrow_down:
...gistry/server/session/push/TriggerPushContext.java 69.23% <0.00%> (+1.92%) :arrow_up:
... and 4 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 24e3f9f...88254e8. Read the comment docs.