Skip to content
This repository has been archived by the owner on May 30, 2022. It is now read-only.

Add load balancer support in annotation #121

Merged
merged 1 commit into from
Nov 24, 2018

Conversation

khotyn
Copy link
Member

@khotyn khotyn commented Nov 24, 2018

Fix issue #120

@khotyn khotyn added the enhancement New feature or request label Nov 24, 2018
@khotyn khotyn added this to the 6.0.1 milestone Nov 24, 2018
@codecov
Copy link

codecov bot commented Nov 24, 2018

Codecov Report

Merging #121 into 6.x will increase coverage by 0.56%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff             @@
##                6.x    #121      +/-   ##
===========================================
+ Coverage     56.23%   56.8%   +0.56%     
- Complexity      402     409       +7     
===========================================
  Files            53      53              
  Lines          1739    1741       +2     
  Branches        311     312       +1     
===========================================
+ Hits            978     989      +11     
+ Misses          579     543      -36     
- Partials        182     209      +27
Impacted Files Coverage Δ Complexity Δ
...pc/boot/runtime/converter/RpcBindingConverter.java 50% <100%> (+0.45%) 28 <0> (+1) ⬆️
...ipay/sofa/rpc/boot/runtime/binding/RpcBinding.java 37.7% <0%> (+1.63%) 14% <0%> (+1%) ⬆️
...y/sofa/rpc/boot/runtime/param/RpcBindingParam.java 44.28% <0%> (+2.85%) 48% <0%> (+3%) ⬆️
...t/runtime/adapter/helper/ConsumerConfigHelper.java 60.55% <0%> (+3.66%) 17% <0%> (+2%) ⬆️

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 16ca9b5...2e117e9. Read the comment docs.

@leizhiyuan leizhiyuan requested review from NeGnail and removed request for NeGnail November 24, 2018 12:27
@leizhiyuan leizhiyuan merged commit 3a44d38 into 6.x Nov 24, 2018
@leizhiyuan leizhiyuan deleted the add_loadbalancer_in_annotation branch November 24, 2018 12:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants