Skip to content

[master] do not show the warning at first replaceWithLazyClient()#8815

Merged
AlbumenJ merged 6 commits intoapache:masterfrom
zrlw:patch-replaceWithLazyClient2
Sep 20, 2021
Merged

[master] do not show the warning at first replaceWithLazyClient()#8815
AlbumenJ merged 6 commits intoapache:masterfrom
zrlw:patch-replaceWithLazyClient2

Conversation

@zrlw
Copy link
Contributor

@zrlw zrlw commented Sep 15, 2021

What is the purpose of the change

fix #8813 on master

@codecov-commenter
Copy link

codecov-commenter commented Sep 15, 2021

Codecov Report

Merging #8815 (e79b690) into master (f8f6995) will decrease coverage by 0.32%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #8815      +/-   ##
============================================
- Coverage     61.05%   60.73%   -0.33%     
+ Complexity      449      447       -2     
============================================
  Files          1100     1100              
  Lines         44422    44422              
  Branches       6469     6468       -1     
============================================
- Hits          27124    26979     -145     
- Misses        14314    14463     +149     
+ Partials       2984     2980       -4     
Impacted Files Coverage Δ
.../rpc/protocol/dubbo/LazyConnectExchangeClient.java 58.24% <100.00%> (+4.39%) ⬆️
...c/protocol/dubbo/ReferenceCountExchangeClient.java 63.33% <100.00%> (ø)
...gistry/client/ServiceDiscoveryRegistryFactory.java 0.00% <0.00%> (-80.00%) ⬇️
...tegration/InterfaceCompatibleRegistryProtocol.java 59.09% <0.00%> (-36.37%) ⬇️
...tadata/DynamicConfigurationServiceNameMapping.java 12.12% <0.00%> (-33.34%) ⬇️
.../org/apache/dubbo/metadata/ServiceNameMapping.java 0.00% <0.00%> (-33.34%) ⬇️
...pache/dubbo/common/function/ThrowableFunction.java 71.42% <0.00%> (-28.58%) ⬇️
...ubbo/registry/client/ServiceDiscoveryRegistry.java 25.88% <0.00%> (-19.42%) ⬇️
...t/migration/DefaultMigrationAddressComparator.java 0.00% <0.00%> (-15.39%) ⬇️
...stry/client/ServiceDiscoveryRegistryDirectory.java 0.00% <0.00%> (-13.53%) ⬇️
... and 22 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 f8f6995...e79b690. Read the comment docs.

@zrlw zrlw changed the title [master] do not warning at first replaceWithLazyClient() [master] do not show the warning at first replaceWithLazyClient() Sep 16, 2021
@AlbumenJ AlbumenJ merged commit e39e494 into apache:master Sep 20, 2021
@zrlw zrlw deleted the patch-replaceWithLazyClient2 branch February 17, 2025 06:30
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.

4 participants

Comments