Skip to content

[type:refactor] refactor rule match cache#4446

Merged
yu199195 merged 36 commits intoapache:masterfrom
moremind:cache-restful-v2
Mar 13, 2023
Merged

[type:refactor] refactor rule match cache#4446
yu199195 merged 36 commits intoapache:masterfrom
moremind:cache-restful-v2

Conversation

@moremind
Copy link
Copy Markdown
Member

@moremind moremind commented Mar 6, 2023

link dashboard pr 👍 apache/shenyu-dashboard#272
Fixes #4419

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:refactor] refactor rule cache [type:refactor] refactor rule match cache Mar 6, 2023
li-keguo
li-keguo previously approved these changes Mar 7, 2023
@li-keguo li-keguo requested a review from yu199195 March 7, 2023 07:02
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 8, 2023

Codecov Report

Merging #4446 (ac37a03) into master (292310a) will increase coverage by 0.04%.
The diff coverage is 61.61%.

@@             Coverage Diff              @@
##             master    #4446      +/-   ##
============================================
+ Coverage     67.93%   67.97%   +0.04%     
- Complexity     7536     7555      +19     
============================================
  Files          1027     1027              
  Lines         29170    29244      +74     
  Branches       2621     2633      +12     
============================================
+ Hits          19816    19879      +63     
- Misses         7791     7794       +3     
- Partials       1563     1571       +8     
Impacted Files Coverage Δ
...ister/AbstractShenyuClientRegisterServiceImpl.java 1.96% <0.00%> (-0.02%) ⬇️
...pache/shenyu/plugin/base/cache/MatchDataCache.java 100.00% <ø> (ø)
...pache/shenyu/plugin/base/AbstractShenyuPlugin.java 60.25% <20.00%> (+15.66%) ⬆️
...in/java/org/apache/shenyu/common/dto/RuleData.java 70.10% <22.22%> (-4.35%) ⬇️
...ava/org/apache/shenyu/common/dto/SelectorData.java 82.29% <22.22%> (-5.35%) ⬇️
...a/org/apache/shenyu/admin/model/entity/RuleDO.java 79.50% <61.53%> (+0.04%) ⬆️
...g/apache/shenyu/admin/model/entity/SelectorDO.java 80.98% <61.53%> (-0.08%) ⬇️
...e/shenyu/web/controller/LocalPluginController.java 94.27% <80.00%> (-0.39%) ⬇️
...ava/org/apache/shenyu/admin/model/dto/RuleDTO.java 82.92% <81.81%> (+0.26%) ⬆️
...org/apache/shenyu/admin/model/dto/SelectorDTO.java 79.16% <81.81%> (+0.51%) ⬆️
... and 4 more

... and 3 files with indirect coverage changes

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

@moremind moremind requested a review from li-keguo March 12, 2023 13:23
@yu199195 yu199195 added this to the 2.6.0 milestone Mar 13, 2023
@yu199195 yu199195 merged commit a7569b4 into apache:master Mar 13, 2023
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.

[Task] ShenYu Level-1 and Level-2 match cache

4 participants