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

增加DubboBootstrap启动停止事件扩展 #7159

Merged
merged 3 commits into from
Feb 7, 2021

Conversation

theonefx
Copy link

增加 DubboBootstrapStartStopListener 来对外通知 DubboBootstarp 的启动停止回调扩展;
向Spring环境抛出DubboBootstarp 的启动停止事件。

#7150

@codecov-io
Copy link

codecov-io commented Feb 1, 2021

Codecov Report

Merging #7159 (6773746) into master (25761bb) will increase coverage by 0.00%.
The diff coverage is 80.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #7159   +/-   ##
=========================================
  Coverage     59.98%   59.99%           
+ Complexity      291      289    -2     
=========================================
  Files          1001     1004    +3     
  Lines         39957    39977   +20     
  Branches       5935     5935           
=========================================
+ Hits          23970    23984   +14     
- Misses        13288    13291    +3     
- Partials       2699     2702    +3     
Impacted Files Coverage Δ Complexity Δ
.../apache/dubbo/config/bootstrap/DubboBootstrap.java 41.93% <66.66%> (+0.24%) 0.00 <0.00> (ø)
...pring/context/event/DubboBootstrapStatedEvent.java 66.66% <66.66%> (ø) 0.00 <0.00> (?)
...pring/context/event/DubboBootstrapStopedEvent.java 66.66% <66.66%> (ø) 0.00 <0.00> (?)
...ing/context/DubboBootstrapApplicationListener.java 100.00% <100.00%> (ø) 0.00 <0.00> (ø)
.../DubboBootstrapStartStopListenerSpringAdapter.java 100.00% <100.00%> (ø) 0.00 <0.00> (?)
...o/remoting/transport/ChannelHandlerDispatcher.java 62.00% <0.00%> (-8.00%) 0.00% <0.00%> (ø%)
...ng/transport/dispatcher/all/AllChannelHandler.java 82.75% <0.00%> (-6.90%) 0.00% <0.00%> (ø%)
...e/dubbo/remoting/transport/netty/NettyChannel.java 55.68% <0.00%> (-4.55%) 20.00% <0.00%> (-1.00%)
...a/org/apache/dubbo/monitor/dubbo/DubboMonitor.java 88.57% <0.00%> (-0.96%) 16.00% <0.00%> (-1.00%)
...ava/org/apache/dubbo/config/ServiceConfigBase.java 58.95% <0.00%> (-0.58%) 0.00% <0.00%> (ø%)
... and 9 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 25761bb...6773746. Read the comment docs.

Copy link
Member

@AlbumenJ AlbumenJ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@@ -46,6 +46,7 @@
public DubboBootstrapApplicationListener(ApplicationContext applicationContext) {
super(applicationContext);
this.dubboBootstrap = DubboBootstrap.getInstance();
DubboBootstrapStartStopListenerSpringAdapter.applicationContext = applicationContext;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a better way to initialize DubboBootstrapStartStopListenerSpringAdapter?

@chickenlj chickenlj changed the base branch from master to 2.7.9-release February 7, 2021 02:39
@chickenlj chickenlj changed the base branch from 2.7.9-release to master February 7, 2021 02:40
@chickenlj chickenlj merged commit d70b151 into apache:master Feb 7, 2021
AlbumenJ added a commit to AlbumenJ/dubbo that referenced this pull request May 26, 2021
@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 80.00000% with 4 lines in your changes missing coverage. Please review.

Project coverage is 59.99%. Comparing base (25761bb) to head (6773746).

Files with missing lines Patch % Lines
.../apache/dubbo/config/bootstrap/DubboBootstrap.java 66.66% 2 Missing ⚠️
...pring/context/event/DubboBootstrapStatedEvent.java 66.66% 1 Missing ⚠️
...pring/context/event/DubboBootstrapStopedEvent.java 66.66% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #7159      +/-   ##
============================================
- Coverage     59.99%   59.99%   -0.01%     
+ Complexity      292      289       -3     
============================================
  Files          1001     1004       +3     
  Lines         39957    39977      +20     
  Branches       5935     5935              
============================================
+ Hits          23973    23984      +11     
- Misses        13287    13291       +4     
- Partials       2697     2702       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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.

6 participants