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:refactor] optimize the performance of MemoryLimitCalculator#maxAvailable #3316

Merged
merged 3 commits into from
Apr 25, 2022

Conversation

loongs-zhang
Copy link
Member

@loongs-zhang loongs-zhang commented Apr 24, 2022

this PR optimize the performance of MemoryLimitCalculator#maxAvailable.

@codecov-commenter
Copy link

codecov-commenter commented Apr 24, 2022

Codecov Report

Merging #3316 (d45b849) into master (dfd5cb3) will decrease coverage by 0.05%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #3316      +/-   ##
============================================
- Coverage     64.40%   64.35%   -0.06%     
+ Complexity     5178     5175       -3     
============================================
  Files           742      742              
  Lines         20957    20964       +7     
  Branches       1919     1919              
============================================
- Hits          13498    13491       -7     
- Misses         6183     6193      +10     
- Partials       1276     1280       +4     
Impacted Files Coverage Δ
...henyu/common/concurrent/MemoryLimitCalculator.java 66.66% <100.00%> (+29.16%) ⬆️
...yu/sync/data/http/refresh/AbstractDataRefresh.java 70.00% <0.00%> (-13.34%) ⬇️
...che/shenyu/sync/data/http/HttpSyncDataService.java 77.23% <0.00%> (-6.51%) ⬇️
...henyu/admin/service/impl/UpstreamCheckService.java 70.14% <0.00%> (-1.50%) ⬇️
...he/shenyu/plugin/logging/AbstractLogCollector.java
...e/shenyu/plugin/logging/LoggingRocketMQPlugin.java
...gin/logging/rocketmq/RocketMQLogCollectClient.java
...shenyu/plugin/logging/entity/ShenyuRequestLog.java
.../apache/shenyu/plugin/logging/sampler/Sampler.java
...he/shenyu/plugin/logging/sampler/CountSampler.java
... and 30 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 dfd5cb3...d45b849. Read the comment docs.

@yu199195 yu199195 merged commit 55cacb0 into apache:master Apr 25, 2022
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.

3 participants