Skip to content

Comments

Fix issue of metadata only registred to one nacos namespace (3.2.x)#13439

Merged
AlbumenJ merged 4 commits intoapache:3.2from
namelessssssssssss:3.2-fix-nacos-metadata
Nov 30, 2023
Merged

Fix issue of metadata only registred to one nacos namespace (3.2.x)#13439
AlbumenJ merged 4 commits intoapache:3.2from
namelessssssssssss:3.2-fix-nacos-metadata

Conversation

@namelessssssssssss
Copy link
Contributor

What is the purpose of the change

Fix issue #13371

nameless and others added 4 commits November 30, 2023 11:50
# Conflicts:
#	dubbo-metadata/dubbo-metadata-api/src/main/java/org/apache/dubbo/metadata/report/MetadataReportInstance.java
#	dubbo-metadata/dubbo-metadata-api/src/main/java/org/apache/dubbo/metadata/report/support/AbstractMetadataReportFactory.java
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

41.7% 41.7% Coverage
0.0% 0.0% Duplication

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (8ac2eaa) 69.47% compared to head (b548ab5) 69.46%.
Report is 1 commits behind head on 3.2.

Additional details and impacted files
@@             Coverage Diff              @@
##                3.2   #13439      +/-   ##
============================================
- Coverage     69.47%   69.46%   -0.02%     
  Complexity        2        2              
============================================
  Files          1653     1653              
  Lines         71634    71645      +11     
  Branches      10270    10271       +1     
============================================
- Hits          49768    49767       -1     
- Misses        17157    17169      +12     
  Partials       4709     4709              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants