[GitHub] zeppelin pull request #2643: fix bugs for paragraph terminated status judgem...

[wary]

GitHub user wary opened a pull request:

    https://github.com/apache/zeppelin/pull/2643

    fix bugs for paragraph terminated status judgement

    ### What is this PR for?
    Fix bugs for paragraph terminated status judgement. 
    The method isTerminate return false forever if a paragraph disabled with 
status ready, if we schedule a note which have a disabled paragraph with status 
ready, cron job will be blocked with the code block：
    
    ```java
    while (!note.isTerminated()) {
            try {
              Thread.sleep(1000);
            } catch (InterruptedException e) {
              logger.error(e.toString(), e);
            }
     }
    ```
    
    ### What type of PR is it?
    [Bug Fix]
    
    ### Todos
    * [ ] - Task
    
    ### What is the Jira issue?
    [ZEPPELIN-3021](https://issues.apache.org/jira/browse/ZEPPELIN-3021)
    
    ### How should this be tested?
    * First time? Setup Travis CI as described on 
https://zeppelin.apache.org/contribution/contributions.html#continuous-integration
    * Strongly recommended: add automated unit tests for any new or changed 
behavior
    * Outline any manual steps to test the PR here.
    
    ### Screenshots (if appropriate)
    
    ### Questions:
    * Does the licenses files need update?
    * Is there breaking changes for older versions?
    * Does this needs documentation?

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/wary/zeppelin FixTerminatedJudgement

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/zeppelin/pull/2643.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 #2643
    
----
commit 863e86ade8f662b4b53bc397ad621a4f35bba271
Author: lei wang <lei1...@outlook.com>
Date:   2017-11-02T01:47:13Z

    fix bugs for paragraph terminated status judgement

----


---