GitHub user zjffdu reopened a pull request:

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

    ZEPPELIN-1516. NPE LivySparkSQLInterpreter thrown with %livy.sql interpreter

    ### What is this PR for?
    
    This PR fixed NPE issue in `LivySparkSQLIntepreter`, also it resolves the 
multiple thread issues discussed in #1503
    ### What type of PR is it?
    
    [Bug Fix]
    ### Todos
    - [ ] - Task
    ### What is the Jira issue?
    - https://issues.apache.org/jira/browse/ZEPPELIN-1516
    ### How should this be tested?
    - Enable shiro
    - Log in 2 users in 2 browsers
    - Start livy sql paragraph at almost the same time
    - 2 livy sessions are created correctly. 
    ### 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-1516

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

    https://github.com/apache/zeppelin/pull/1514.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 #1514
    
----
commit 0b5825c6769ecb3382c4d2569380fde70d8a032b
Author: Jeff Zhang <zjf...@apache.org>
Date:   2016-10-14T01:27:52Z

    ZEPPELIN-1516. NPE LivySparkSQLInterpreter thrown with %livy.sql interpreter

commit fc5692df27ed4d2313b0c0d43bcc26f11efab5f3
Author: Jeff Zhang <zjf...@apache.org>
Date:   2016-10-27T05:38:03Z

    add test

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to