Skip to content

Improve cache init time and add application name #2420

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

Merged
merged 5 commits into from
Sep 13, 2023
Merged

Improve cache init time and add application name #2420

merged 5 commits into from
Sep 13, 2023

Conversation

FinalT
Copy link
Member

@FinalT FinalT commented Sep 5, 2023

No description provided.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 5, 2023

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

No Coverage information No Coverage information
0.0% 0.0% Duplication

@codecov-commenter
Copy link

Codecov Report

Merging #2420 (88a1eb3) into main (4b9bbd8) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

❗ Current head 88a1eb3 differs from pull request most recent head d5b21cc. Consider uploading reports for the commit d5b21cc to get more accurate results

@@            Coverage Diff             @@
##             main    #2420      +/-   ##
==========================================
- Coverage   45.04%   45.03%   -0.02%     
==========================================
  Files         269      269              
  Lines       18163    18167       +4     
==========================================
- Hits         8182     8181       -1     
- Misses       9109     9116       +7     
+ Partials      872      870       -2     
Files Changed Coverage Δ
...scovery/service_instances_changed_listener_impl.go 0.00% <0.00%> (-2.35%) ⬇️

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Contributor

@chickenlj chickenlj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Copy link
Contributor

@chickenlj chickenlj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@chickenlj chickenlj merged commit d10057a into apache:main Sep 13, 2023
"dubbo.apache.org/dubbo-go/v3/registry"
"dubbo.apache.org/dubbo-go/v3/registry/servicediscovery/store"
"dubbo.apache.org/dubbo-go/v3/remoting"
)

var (
metaCache *store.CacheManager
one sync.Once
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

改下名字吧,cacheOnce

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed, 'once' or 'XxxOnce' should be a better name.

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.

4 participants