GitHub user lvicentesanchez opened a pull request:

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

    [ZEPPELIN-2900] Close connection on getUserList

    ### What is this PR for?
    
    This PR fixed a JDBC connection leak on `GetUserList.getUserList` when the 
Shiro Realm is a JdbcRealm. After a while, it's not possible to even login into 
zeppelin.
    
    ### What type of PR is it?
    
    Bug Fix
    
    ### Todos
    
    N/A
    
    ### What is the Jira issue?
    
    [ZEPPELIN-2900](https://issues.apache.org/jira/browse/ZEPPELIN-2900)
    
    ### How should this be tested?
    
    In order to reproduce the bug follow this steps:
    
    1. Configure zeppelin to use a JdbcRealm for authentication with a low 
number of connections in the connection pool.
    2. Login into zeppelin and open a notebook.
    3. Open the permissions form of the notebook.
    4. Try to add several users to the owners list (more than the number of 
connections of the connection pool.
    
    ### Screenshots (if appropriate)
    
    ### Questions:
    * Does the licenses files need update? No.
    * Is there breaking changes for older versions? No that I'm aware.
    * Does this needs documentation? No.


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

    $ git pull https://github.com/bigcentech/zeppelin bugfix/zeppelin-2900

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

    https://github.com/apache/zeppelin/pull/2567.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 #2567
    
----
commit da24668aafed9135a360ade5be195c9eccccc550
Author: Luis Vicente <langel.vice...@gmail.com>
Date:   2017-09-05T10:59:56Z

    [ZEPPELIN-2900] Close connection on getUserList

----


---

Reply via email to