GitHub user zjffdu opened a pull request:
https://github.com/apache/zeppelin/pull/2858
ZEPPELIN-3315. Merge beforeStatusChange and afterStatusChange to
onStatusChange
### What is this PR for?
The signature of `beforeStatusChange` & `afterStatusChange` include both
the status of before and after, so it is not necessary to create both
`beforeStatusChange` & `afterStatusChange`. Only one method `onStatusChange` is
sufficient.
### What type of PR is it?
[Refactoring]
### Todos
* [ ] - Task
### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-3315
### How should this be tested?
* CI pass
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/zjffdu/zeppelin ZEPPELIN-3315
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zeppelin/pull/2858.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2858
----
commit f0fe59455cdcb239dab9213b9ed21c1105f61f51
Author: Jeff Zhang <zjffdu@...>
Date: 2018-03-12T02:12:51Z
ZEPPELIN-3315. Merge beforeStatusChange and afterStatusChange to
onStatusChange
----
---
