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

[ISSUE #4431] add p2cLoadBalancer #4451

Merged
merged 3 commits into from
Mar 9, 2023
Merged

Conversation

HaiqiQin
Copy link
Contributor

@HaiqiQin HaiqiQin commented Mar 8, 2023

#4431
add p2cLoadBalancer

@codecov-commenter
Copy link

codecov-commenter commented Mar 9, 2023

Codecov Report

Merging #4451 (b33e200) into master (8695376) will increase coverage by 0.00%.
The diff coverage is 77.61%.

@@            Coverage Diff            @@
##             master    #4451   +/-   ##
=========================================
  Coverage     68.05%   68.06%           
- Complexity     7541     7553   +12     
=========================================
  Files          1026     1027    +1     
  Lines         29101    29166   +65     
  Branches       2611     2621   +10     
=========================================
+ Hits          19806    19853   +47     
- Misses         7739     7749   +10     
- Partials       1556     1564    +8     
Impacted Files Coverage Δ
...rg/apache/shenyu/loadbalancer/entity/Upstream.java 92.77% <61.53%> (-5.81%) ⬇️
.../org/apache/shenyu/plugin/divide/DividePlugin.java 60.29% <68.42%> (+2.29%) ⬆️
...pache/shenyu/loadbalancer/spi/P2cLoadBalancer.java 87.87% <87.87%> (ø)
...rg/apache/shenyu/common/enums/LoadBalanceEnum.java 100.00% <100.00%> (ø)
...henyu/admin/service/impl/UpstreamCheckService.java 66.28% <0.00%> (-1.15%) ⬇️
...henyu/plugin/grpc/resolver/ShenyuNameResolver.java 60.63% <0.00%> (-1.07%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@@ -1704,6 +1704,7 @@ INSERT INTO `shenyu_dict` VALUES ('1529402613195784193', 'loadBalance', 'LOAD_BA
INSERT INTO `shenyu_dict` VALUES ('1529402613195784194', 'loadBalance', 'LOAD_BALANCE', 'random', 'random', 'random', 1, 1, '2022-05-25 18:02:52', '2022-05-25 18:02:52');
INSERT INTO `shenyu_dict` VALUES ('1529402613195784195', 'loadBalance', 'LOAD_BALANCE', 'hash', 'hash', 'hash', 0, 1, '2022-05-25 18:02:52', '2022-05-25 18:02:52');
INSERT INTO `shenyu_dict` VALUES ('1572621976689762307', 'loadBalance', 'LOAD_BALANCE', 'leastActive', 'leastActive', 'leastActive', 3, 1, '2023-01-17 18:02:52', '2023-01-17 18:02:52');
INSERT INTO `shenyu_dict` VALUES ('1572621976689762308', 'loadBalance', 'LOAD_BALANCE', 'p2c', 'p2c', 'p2c', 4, 1, '2023-03-07 22:12:12', '2023-03-07 22:12:12');
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe add upgrade sql script?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i will add.

@yu199195 yu199195 added this to the 2.6.0 milestone Mar 9, 2023
@yu199195 yu199195 merged commit 1363524 into apache:master Mar 9, 2023
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.

None yet

3 participants