Skip to content

add leastActive load balance#4326

Merged
yu199195 merged 6 commits intoapache:masterfrom
mahaitao617:mht_strategy_116
Jan 17, 2023
Merged

add leastActive load balance#4326
yu199195 merged 6 commits intoapache:masterfrom
mahaitao617:mht_strategy_116

Conversation

@mahaitao617
Copy link
Copy Markdown
Contributor

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jan 17, 2023

Codecov Report

Merging #4326 (7e99092) into master (c1408e5) will increase coverage by 0.00%.
The diff coverage is 62.50%.

@@            Coverage Diff            @@
##             master    #4326   +/-   ##
=========================================
  Coverage     68.04%   68.05%           
- Complexity     7484     7486    +2     
=========================================
  Files          1019     1020    +1     
  Lines         28796    28819   +23     
  Branches       2579     2581    +2     
=========================================
+ Hits          19595    19612   +17     
- Misses         7672     7675    +3     
- Partials       1529     1532    +3     
Impacted Files Coverage Δ
...henyu/loadbalancer/spi/LeastActiveLoadBalance.java 59.09% <59.09%> (ø)
...rg/apache/shenyu/common/enums/LoadBalanceEnum.java 100.00% <100.00%> (ø)
...shenyu/sync/data/consul/ConsulSyncDataService.java 78.33% <0.00%> (-1.67%) ⬇️
...ruptor/RegisterClientServerDisruptorPublisher.java 66.66% <0.00%> (+11.11%) ⬆️
...controller/ShenyuClientHttpRegistryController.java 81.81% <0.00%> (+18.18%) ⬆️

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

@yu199195 yu199195 added this to the 2.5.1 milestone Jan 17, 2023
@yu199195 yu199195 merged commit c195af0 into apache:master Jan 17, 2023
@mahaitao617 mahaitao617 deleted the mht_strategy_116 branch January 17, 2023 05:57
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