trpc-group / trpc-cpp

A pluggable, high-performance RPC framework written in cpp
Other
270 stars 78 forks source link

Optimize: optimize client class or datastruct, read wait-free #127

Closed helloopenworld closed 5 months ago

helloopenworld commented 5 months ago
codecov[bot] commented 5 months ago

Codecov Report

Attention: Patch coverage is 83.71041% with 36 lines in your changes are missing coverage. Please review.

Project coverage is 81.65658%. Comparing base (ce52448) to head (ab4e296).

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/trpc-group/trpc-cpp/pull/127/graphs/tree.svg?width=650&height=150&src=pr&token=7csQnBLsa9&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=trpc-group)](https://app.codecov.io/gh/trpc-group/trpc-cpp/pull/127?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=trpc-group) ```diff @@ Coverage Diff @@ ## main #127 +/- ## =================================================== - Coverage 81.69858% 81.65658% -0.04201% =================================================== Files 855 856 +1 Lines 33899 34034 +135 =================================================== + Hits 27695 27791 +96 - Misses 6204 6243 +39 ``` | [Files](https://app.codecov.io/gh/trpc-group/trpc-cpp/pull/127?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=trpc-group) | Coverage Δ | | |---|---|---| | [trpc/client/service\_proxy.cc](https://app.codecov.io/gh/trpc-group/trpc-cpp/pull/127?src=pr&el=tree&filepath=trpc%2Fclient%2Fservice_proxy.cc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=trpc-group#diff-dHJwYy9jbGllbnQvc2VydmljZV9wcm94eS5jYw==) | `77.92553% <100.00000%> (+0.05885%)` | :arrow_up: | | [trpc/client/service\_proxy.h](https://app.codecov.io/gh/trpc-group/trpc-cpp/pull/127?src=pr&el=tree&filepath=trpc%2Fclient%2Fservice_proxy.h&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=trpc-group#diff-dHJwYy9jbGllbnQvc2VydmljZV9wcm94eS5o) | `100.00000% <ø> (ø)` | | | [trpc/client/service\_proxy\_manager.cc](https://app.codecov.io/gh/trpc-group/trpc-cpp/pull/127?src=pr&el=tree&filepath=trpc%2Fclient%2Fservice_proxy_manager.cc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=trpc-group#diff-dHJwYy9jbGllbnQvc2VydmljZV9wcm94eV9tYW5hZ2VyLmNj) | `98.27586% <100.00000%> (-0.02923%)` | :arrow_down: | | [trpc/client/service\_proxy\_option\_setter.cc](https://app.codecov.io/gh/trpc-group/trpc-cpp/pull/127?src=pr&el=tree&filepath=trpc%2Fclient%2Fservice_proxy_option_setter.cc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=trpc-group#diff-dHJwYy9jbGllbnQvc2VydmljZV9wcm94eV9vcHRpb25fc2V0dGVyLmNj) | `98.21429% <100.00000%> (+0.03247%)` | :arrow_up: | | [trpc/common/config/client\_conf.h](https://app.codecov.io/gh/trpc-group/trpc-cpp/pull/127?src=pr&el=tree&filepath=trpc%2Fcommon%2Fconfig%2Fclient_conf.h&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=trpc-group#diff-dHJwYy9jb21tb24vY29uZmlnL2NsaWVudF9jb25mLmg=) | `100.00000% <ø> (ø)` | | | [trpc/common/config/client\_conf\_parser.h](https://app.codecov.io/gh/trpc-group/trpc-cpp/pull/127?src=pr&el=tree&filepath=trpc%2Fcommon%2Fconfig%2Fclient_conf_parser.h&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=trpc-group#diff-dHJwYy9jb21tb24vY29uZmlnL2NsaWVudF9jb25mX3BhcnNlci5o) | `100.00000% <100.00000%> (ø)` | | | [trpc/common/trpc\_plugin.cc](https://app.codecov.io/gh/trpc-group/trpc-cpp/pull/127?src=pr&el=tree&filepath=trpc%2Fcommon%2Ftrpc_plugin.cc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=trpc-group#diff-dHJwYy9jb21tb24vdHJwY19wbHVnaW4uY2M=) | `58.40000% <100.00000%> (+0.16706%)` | :arrow_up: | | [trpc/server/non\_rpc/non\_rpc\_service\_impl.cc](https://app.codecov.io/gh/trpc-group/trpc-cpp/pull/127?src=pr&el=tree&filepath=trpc%2Fserver%2Fnon_rpc%2Fnon_rpc_service_impl.cc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=trpc-group#diff-dHJwYy9zZXJ2ZXIvbm9uX3JwYy9ub25fcnBjX3NlcnZpY2VfaW1wbC5jYw==) | `100.00000% <ø> (ø)` | | | [...n\_complex/fiber\_tcp\_conn\_complex\_connector\_group.h](https://app.codecov.io/gh/trpc-group/trpc-cpp/pull/127?src=pr&el=tree&filepath=trpc%2Ftransport%2Fclient%2Ffiber%2Fconn_complex%2Ffiber_tcp_conn_complex_connector_group.h&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=trpc-group#diff-dHJwYy90cmFuc3BvcnQvY2xpZW50L2ZpYmVyL2Nvbm5fY29tcGxleC9maWJlcl90Y3BfY29ubl9jb21wbGV4X2Nvbm5lY3Rvcl9ncm91cC5o) | `100.00000% <ø> (ø)` | | | [...sport/client/fiber/fiber\_connector\_group\_manager.h](https://app.codecov.io/gh/trpc-group/trpc-cpp/pull/127?src=pr&el=tree&filepath=trpc%2Ftransport%2Fclient%2Ffiber%2Ffiber_connector_group_manager.h&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=trpc-group#diff-dHJwYy90cmFuc3BvcnQvY2xpZW50L2ZpYmVyL2ZpYmVyX2Nvbm5lY3Rvcl9ncm91cF9tYW5hZ2VyLmg=) | `100.00000% <ø> (ø)` | | | ... and [8 more](https://app.codecov.io/gh/trpc-group/trpc-cpp/pull/127?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=trpc-group) | | ... and [8 files with indirect coverage changes](https://app.codecov.io/gh/trpc-group/trpc-cpp/pull/127/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=trpc-group)
chhy2009 commented 5 months ago

lgtm