Skip to content

add auto-generated Regsitry and Protocol to ConfigManager#5516

Merged
chickenlj merged 3 commits intoapache:2.7.5-releasefrom
chickenlj:fix-default-protocol&ergistry
Dec 20, 2019
Merged

add auto-generated Regsitry and Protocol to ConfigManager#5516
chickenlj merged 3 commits intoapache:2.7.5-releasefrom
chickenlj:fix-default-protocol&ergistry

Conversation

@chickenlj
Copy link
Copy Markdown
Contributor

When user does not specify any ProtocolConfig or RegistryConfig, the auto-generated Regsitry and Protocol should be added to ConfigManager

@CodingSinger
Copy link
Copy Markdown
Member

LGTM

@codecov-io
Copy link
Copy Markdown

codecov-io commented Dec 20, 2019

Codecov Report

Merging #5516 into 2.7.5-release will decrease coverage by 0.05%.
The diff coverage is 100%.

Impacted file tree graph

@@                 Coverage Diff                 @@
##             2.7.5-release    #5516      +/-   ##
===================================================
- Coverage            61.08%   61.03%   -0.06%     
+ Complexity             423      421       -2     
===================================================
  Files                  921      921              
  Lines                37588    37590       +2     
  Branches              5465     5465              
===================================================
- Hits                 22961    22943      -18     
- Misses               12120    12137      +17     
- Partials              2507     2510       +3
Impacted Files Coverage Δ Complexity Δ
...ava/org/apache/dubbo/config/ServiceConfigBase.java 57% <100%> (+0.43%) 0 <0> (ø) ⬇️
...bo/config/event/listener/LoggingEventListener.java 62.5% <0%> (ø) 0% <0%> (ø) ⬇️
...ache/dubbo/remoting/transport/AbstractChannel.java 87.5% <0%> (ø) 0% <0%> (ø) ⬇️
...ava/org/apache/dubbo/config/DubboShutdownHook.java 59.09% <0%> (ø) 0% <0%> (ø) ⬇️
...che/dubbo/remoting/transport/mina/MinaChannel.java 43.42% <0%> (-10.53%) 16% <0%> (-1%)
...mmon/threadpool/support/fixed/FixedThreadPool.java 87.5% <0%> (ø) 0% <0%> (ø) ⬇️
...ng/transport/dispatcher/all/AllChannelHandler.java 62.06% <0%> (-6.9%) 0% <0%> (ø)
.../threadpool/support/limited/LimitedThreadPool.java 88.88% <0%> (ø) 0% <0%> (ø) ⬇️
...ng/exchange/support/header/HeartbeatTimerTask.java 78.94% <0%> (ø) 0% <0%> (ø) ⬇️
.../org/apache/dubbo/remoting/ExecutionException.java 15.78% <0%> (-5.27%) 0% <0%> (ø)
... and 33 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 6249f0f...4a176aa. Read the comment docs.

@chickenlj chickenlj merged commit 59bc99e into apache:2.7.5-release Dec 20, 2019
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.

3 participants