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

Fix the issue of taking the zone parameter value in ZoneAwareClusterInvoker #8521

Merged
merged 1 commit into from Aug 23, 2021

Conversation

BurningCN
Copy link
Contributor

@BurningCN BurningCN commented Aug 16, 2021

What is the purpose of the change

issue #8451

@codecov-commenter
Copy link

@codecov-commenter codecov-commenter commented Aug 16, 2021

Codecov Report

Merging #8521 (0b10e14) into master (6acd5a2) will decrease coverage by 0.74%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #8521      +/-   ##
============================================
- Coverage     61.06%   60.31%   -0.75%     
+ Complexity      447      446       -1     
============================================
  Files          1097     1097              
  Lines         44401    44354      -47     
  Branches       6469     6437      -32     
============================================
- Hits          27113    26754     -359     
- Misses        14302    14626     +324     
+ Partials       2986     2974      -12     
Impacted Files Coverage Δ
...ster/support/registry/ZoneAwareClusterInvoker.java 0.00% <0.00%> (ø)
...gcenter/wrapper/CompositeDynamicConfiguration.java 0.00% <0.00%> (-73.34%) ⬇️
...ookeeper/ZookeeperDynamicConfigurationFactory.java 0.00% <0.00%> (-66.67%) ⬇️
...pport/zookeeper/ZookeeperDynamicConfiguration.java 0.00% <0.00%> (-62.86%) ⬇️
...fig/configcenter/TreePathDynamicConfiguration.java 31.42% <0.00%> (-54.29%) ⬇️
.../dubbo/metadata/report/MetadataReportInstance.java 12.50% <0.00%> (-40.63%) ⬇️
.../configcenter/support/zookeeper/CacheListener.java 0.00% <0.00%> (-34.29%) ⬇️
...gistry/client/migration/MigrationRuleListener.java 55.88% <0.00%> (-23.53%) ⬇️
...ng/transport/dispatcher/all/AllChannelHandler.java 62.06% <0.00%> (-20.69%) ⬇️
...mmon/config/configcenter/DynamicConfiguration.java 68.18% <0.00%> (-18.19%) ⬇️
... and 57 more

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 6acd5a2...0b10e14. Read the comment docs.

@AlbumenJ AlbumenJ merged commit c22d746 into apache:master Aug 23, 2021
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants