-
Notifications
You must be signed in to change notification settings - Fork 26.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Configuration center metrics separate module #12096
Configuration center metrics separate module #12096
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to modify a few small areas
...cos/src/main/java/org/apache/dubbo/configcenter/support/nacos/NacosDynamicConfiguration.java
Outdated
Show resolved
Hide resolved
...per/src/main/java/org/apache/dubbo/configcenter/support/zookeeper/ZookeeperDataListener.java
Outdated
Show resolved
Hide resolved
…bbo into config-center-metrics
…bbo into config-center-metrics
b8ae411
to
0095068
Compare
Codecov Report
@@ Coverage Diff @@
## 3.2 #12096 +/- ##
============================================
- Coverage 69.80% 69.56% -0.24%
+ Complexity 340 2 -338
============================================
Files 3399 1592 -1807
Lines 159278 65390 -93888
Branches 26524 9520 -17004
============================================
- Hits 111185 45490 -65695
+ Misses 38390 15526 -22864
+ Partials 9703 4374 -5329 see 1843 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Kudos, SonarCloud Quality Gate passed! |
What is the purpose of the change
Detach from default package, use public event api