Skip to content
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

[type:fix] fix cache exist empty data #4716

Merged
merged 2 commits into from Jun 9, 2023

Conversation

moremind
Copy link
Member

@moremind moremind commented Jun 8, 2023

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

@moremind moremind changed the title [type:feat] refactor cache [type:fix] fix cache exist empty data Jun 8, 2023
@codecov-commenter
Copy link

codecov-commenter commented Jun 8, 2023

Codecov Report

Merging #4716 (b8a6f3d) into master (41d511b) will decrease coverage by 0.11%.
The diff coverage is 47.36%.

@@             Coverage Diff              @@
##             master    #4716      +/-   ##
============================================
- Coverage     66.50%   66.40%   -0.11%     
+ Complexity     8045     8035      -10     
============================================
  Files          1111     1111              
  Lines         32166    32183      +17     
  Branches       2905     2907       +2     
============================================
- Hits          21391    21370      -21     
- Misses         9079     9112      +33     
- Partials       1696     1701       +5     
Impacted Files Coverage Δ
...plugin/springcloud/cache/ServiceInstanceCache.java 16.66% <0.00%> (-3.34%) ⬇️
...pache/shenyu/plugin/base/cache/MatchDataCache.java 65.30% <40.00%> (-6.49%) ⬇️
.../plugin/base/cache/CommonPluginDataSubscriber.java 64.00% <100.00%> (+1.24%) ⬆️

... and 9 files with indirect coverage changes

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

@loongs-zhang loongs-zhang merged commit 3207bb1 into apache:master Jun 9, 2023
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants