Skip to content

[type:feature]Http sync, add property.#2736

Merged
yu199195 merged 1 commit intoapache:masterfrom
KevinClair:httpsync
Jan 9, 2022
Merged

[type:feature]Http sync, add property.#2736
yu199195 merged 1 commit intoapache:masterfrom
KevinClair:httpsync

Conversation

@KevinClair
Copy link
Copy Markdown
Member

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 mvn clean install -Dmaven.javadoc.skip=true.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jan 9, 2022

Codecov Report

Merging #2736 (9dbf267) into master (09768be) will decrease coverage by 0.08%.
The diff coverage is 25.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #2736      +/-   ##
============================================
- Coverage     60.26%   60.17%   -0.09%     
+ Complexity     4409     4402       -7     
============================================
  Files           681      681              
  Lines         19577    19587      +10     
  Branches       1735     1740       +5     
============================================
- Hits          11798    11787      -11     
- Misses         6786     6803      +17     
- Partials        993      997       +4     
Impacted Files Coverage Δ
...g/apache/shenyu/common/constant/HttpConstants.java 0.00% <0.00%> (ø)
...che/shenyu/sync/data/http/HttpSyncDataService.java 77.94% <25.00%> (+3.67%) ⬆️
...pache/shenyu/sync/data/http/config/HttpConfig.java 70.58% <30.00%> (-14.03%) ⬇️
...henyu/plugin/grpc/resolver/ShenyuNameResolver.java 62.76% <0.00%> (-9.58%) ⬇️
.../plugin/grpc/loadbalance/AbstractLoadBalancer.java 70.65% <0.00%> (-8.70%) ⬇️
...henyu/admin/service/impl/UpstreamCheckService.java 63.49% <0.00%> (-0.80%) ⬇️

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 09768be...9dbf267. Read the comment docs.

@yu199195 yu199195 added this to the 2.4.2 milestone Jan 9, 2022
@yu199195 yu199195 merged commit 09ddd3f into apache:master Jan 9, 2022
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