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

Add cache mechanism for LoadRules in isolation module #322

Merged
merged 2 commits into from
Nov 16, 2020

Conversation

binbin0325
Copy link
Collaborator

Describe what this PR does / why we need it
In order to avoid the meaningless updates to the property, downstream of property manager should
cache last update value to check consistency.

Does this pull request fix one issue?
Fix:#111

Describe how you did it
Using reflect.deepequal,maybe we need to think about performance

Describe how to verify it
ut

Special notes for reviews

@codecov-io
Copy link

codecov-io commented Nov 11, 2020

Codecov Report

Merging #322 (8f4f43c) into master (14fe759) will increase coverage by 0.27%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #322      +/-   ##
==========================================
+ Coverage   50.41%   50.68%   +0.27%     
==========================================
  Files          68       68              
  Lines        3709     3774      +65     
==========================================
+ Hits         1870     1913      +43     
- Misses       1577     1596      +19     
- Partials      262      265       +3     
Impacted Files Coverage Δ
core/isolation/rule_manager.go 63.76% <100.00%> (+4.09%) ⬆️
core/circuitbreaker/slot.go 94.11% <0.00%> (-5.89%) ⬇️
ext/datasource/helper.go 79.19% <0.00%> (-2.78%) ⬇️
core/hotspot/rule_manager.go 51.76% <0.00%> (-1.90%) ⬇️
core/flow/slot.go 43.33% <0.00%> (-1.50%) ⬇️
core/system/slot.go 53.06% <0.00%> (-1.11%) ⬇️
core/hotspot/slot.go 25.00% <0.00%> (-0.72%) ⬇️
core/base/slot_chain.go 97.70% <0.00%> (ø)
core/hotspot/concurrency_stat_slot.go 6.25% <0.00%> (+2.54%) ⬆️
core/flow/standalone_stat_slot.go 75.00% <0.00%> (+3.57%) ⬆️
... and 3 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 14fe759...8f4f43c. Read the comment docs.

@louyuting louyuting self-requested a review November 14, 2020 14:47
@louyuting louyuting added the area/flow-isolation Issues or PRs related to isolation (concurrency limiting) label Nov 14, 2020
@louyuting louyuting added this to the 1.0.0 milestone Nov 14, 2020
@louyuting louyuting force-pushed the isolation_rule_cache branch from 8f4f43c to e90dc22 Compare November 16, 2020 13:07
@louyuting louyuting force-pushed the isolation_rule_cache branch from e90dc22 to 15d6fd5 Compare November 16, 2020 13:10
@louyuting louyuting merged commit 5bcbcc5 into alibaba:master Nov 16, 2020
@louyuting louyuting changed the title Loading isolation rule supports cache Add cache mechanism for LoadRules in isolation module Nov 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/flow-isolation Issues or PRs related to isolation (concurrency limiting)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants