Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[ISSUE #1239]🔥Optimize KVConfigManager code⚡️ #1240

Merged
merged 1 commit into from
Nov 20, 2024

[ISSUE #1239]🔥Optimize KVConfigManager code⚡️

4a094be
Select commit
Loading
Failed to load commit list.
Merged

[ISSUE #1239]🔥Optimize KVConfigManager code⚡️ #1240

[ISSUE #1239]🔥Optimize KVConfigManager code⚡️
4a094be
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Nov 20, 2024 in 0s

18.26% (+0.23%) compared to 3998417

View this Pull Request on Codecov

18.26% (+0.23%) compared to 3998417

Details

Codecov Report

Attention: Patch coverage is 72.13115% with 34 lines in your changes missing coverage. Please review.

Project coverage is 18.26%. Comparing base (3998417) to head (4a094be).

Files with missing lines Patch % Lines
...namesrv/src/processor/default_request_processor.rs 0.00% 22 Missing ⚠️
...-namesrv/src/processor/client_request_processor.rs 0.00% 6 Missing ⚠️
rocketmq-namesrv/src/kvconfig/kvconfig_mananger.rs 96.70% 3 Missing ⚠️
rocketmq-namesrv/src/bootstrap.rs 0.00% 2 Missing ⚠️
rocketmq-namesrv/src/route/route_info_manager.rs 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1240      +/-   ##
==========================================
+ Coverage   18.02%   18.26%   +0.23%     
==========================================
  Files         428      428              
  Lines       53563    53595      +32     
==========================================
+ Hits         9655     9789     +134     
+ Misses      43908    43806     -102     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


🚨 Try these New Features: