You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "gaojinchao (Created) (JIRA)" <ji...@apache.org> on 2011/11/11 10:24:51 UTC

[jira] [Created] (HBASE-4773) HBaseAdmin leaks ZooKeeper connections

HBaseAdmin leaks ZooKeeper connections
--------------------------------------

                 Key: HBASE-4773
                 URL: https://issues.apache.org/jira/browse/HBASE-4773
             Project: HBase
          Issue Type: Bug
          Components: client
    Affects Versions: 0.90.4
            Reporter: gaojinchao
            Priority: Critical
             Fix For: 0.90.5


When master crashs, HBaseAdmin will leaks ZooKeeper connections
I think we should close the zk connetion when throw MasterNotRunningException

 public HBaseAdmin(Configuration c)
  throws MasterNotRunningException, ZooKeeperConnectionException {
    this.conf = HBaseConfiguration.create(c);
    this.connection = HConnectionManager.getConnection(this.conf);
    this.pause = this.conf.getLong("hbase.client.pause", 1000);
    this.numRetries = this.conf.getInt("hbase.client.retries.number", 10);
    this.retryLongerMultiplier = this.conf.getInt("hbase.client.retries.longer.multiplier", 10);

    //we should add this code and close the zk connection
    try{
      this.connection.getMaster();
    }catch(MasterNotRunningException e){
      HConnectionManager.deleteConnection(conf, false);
      throw e;      
    }
  }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-4773) HBaseAdmin leaks ZooKeeper connections

Posted by "xufeng (Updated) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-4773?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

xufeng updated HBASE-4773:
--------------------------

    Attachment: 4773.patch

created the branches patch, here are test result:

Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 27.675 sec
Running org.apache.hadoop.hbase.TestScanMultipleVersions
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 24.155 sec
Running org.apache.hadoop.hbase.rest.model.TestStorageClusterVersionModel
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.026 sec
Running org.apache.hadoop.hbase.client.TestHCM
Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 43.686 sec

Results :

Tests run: 694, Failures: 0, Errors: 0, Skipped: 9
                
> HBaseAdmin leaks ZooKeeper connections
> --------------------------------------
>
>                 Key: HBASE-4773
>                 URL: https://issues.apache.org/jira/browse/HBASE-4773
>             Project: HBase
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 0.90.4
>            Reporter: gaojinchao
>            Priority: Critical
>             Fix For: 0.90.5
>
>         Attachments: 4773.patch
>
>
> When master crashs, HBaseAdmin will leaks ZooKeeper connections
> I think we should close the zk connetion when throw MasterNotRunningException
>  public HBaseAdmin(Configuration c)
>   throws MasterNotRunningException, ZooKeeperConnectionException {
>     this.conf = HBaseConfiguration.create(c);
>     this.connection = HConnectionManager.getConnection(this.conf);
>     this.pause = this.conf.getLong("hbase.client.pause", 1000);
>     this.numRetries = this.conf.getInt("hbase.client.retries.number", 10);
>     this.retryLongerMultiplier = this.conf.getInt("hbase.client.retries.longer.multiplier", 10);
>     //we should add this code and close the zk connection
>     try{
>       this.connection.getMaster();
>     }catch(MasterNotRunningException e){
>       HConnectionManager.deleteConnection(conf, false);
>       throw e;      
>     }
>   }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-4773) HBaseAdmin may leak ZooKeeper connections

Posted by "Hudson (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-4773?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13159780#comment-13159780 ] 

Hudson commented on HBASE-4773:
-------------------------------

Integrated in HBase-0.92 #163 (See [https://builds.apache.org/job/HBase-0.92/163/])
    HBASE-4773  HBaseAdmin may leak ZooKeeper connections (Xufeng)

tedyu : 
Files : 
* /hbase/branches/0.92/CHANGES.txt
* /hbase/branches/0.92/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java

                
> HBaseAdmin may leak ZooKeeper connections
> -----------------------------------------
>
>                 Key: HBASE-4773
>                 URL: https://issues.apache.org/jira/browse/HBASE-4773
>             Project: HBase
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 0.90.4
>            Reporter: gaojinchao
>            Assignee: xufeng
>            Priority: Critical
>             Fix For: 0.90.5
>
>         Attachments: 4773.patch, branches_4773.patch, trunk_4773_patch.patch
>
>
> When master crashs, HBaseAdmin will leaks ZooKeeper connections
> I think we should close the zk connetion when throw MasterNotRunningException
>  public HBaseAdmin(Configuration c)
>   throws MasterNotRunningException, ZooKeeperConnectionException {
>     this.conf = HBaseConfiguration.create(c);
>     this.connection = HConnectionManager.getConnection(this.conf);
>     this.pause = this.conf.getLong("hbase.client.pause", 1000);
>     this.numRetries = this.conf.getInt("hbase.client.retries.number", 10);
>     this.retryLongerMultiplier = this.conf.getInt("hbase.client.retries.longer.multiplier", 10);
>     //we should add this code and close the zk connection
>     try{
>       this.connection.getMaster();
>     }catch(MasterNotRunningException e){
>       HConnectionManager.deleteConnection(conf, false);
>       throw e;      
>     }
>   }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-4773) HBaseAdmin leaks ZooKeeper connections

Posted by "Ted Yu (Updated) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-4773?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ted Yu updated HBASE-4773:
--------------------------

    Status: Patch Available  (was: Open)
    
> HBaseAdmin leaks ZooKeeper connections
> --------------------------------------
>
>                 Key: HBASE-4773
>                 URL: https://issues.apache.org/jira/browse/HBASE-4773
>             Project: HBase
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 0.90.4
>            Reporter: gaojinchao
>            Priority: Critical
>             Fix For: 0.90.5
>
>         Attachments: 4773.patch, branches_4773.patch, trunk_4773_patch.patch
>
>
> When master crashs, HBaseAdmin will leaks ZooKeeper connections
> I think we should close the zk connetion when throw MasterNotRunningException
>  public HBaseAdmin(Configuration c)
>   throws MasterNotRunningException, ZooKeeperConnectionException {
>     this.conf = HBaseConfiguration.create(c);
>     this.connection = HConnectionManager.getConnection(this.conf);
>     this.pause = this.conf.getLong("hbase.client.pause", 1000);
>     this.numRetries = this.conf.getInt("hbase.client.retries.number", 10);
>     this.retryLongerMultiplier = this.conf.getInt("hbase.client.retries.longer.multiplier", 10);
>     //we should add this code and close the zk connection
>     try{
>       this.connection.getMaster();
>     }catch(MasterNotRunningException e){
>       HConnectionManager.deleteConnection(conf, false);
>       throw e;      
>     }
>   }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-4773) HBaseAdmin leaks ZooKeeper connections

Posted by "ramkrishna.s.vasudevan (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-4773?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13157183#comment-13157183 ] 

ramkrishna.s.vasudevan commented on HBASE-4773:
-----------------------------------------------

+1 on patch
                
> HBaseAdmin leaks ZooKeeper connections
> --------------------------------------
>
>                 Key: HBASE-4773
>                 URL: https://issues.apache.org/jira/browse/HBASE-4773
>             Project: HBase
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 0.90.4
>            Reporter: gaojinchao
>            Priority: Critical
>             Fix For: 0.90.5
>
>         Attachments: 4773.patch
>
>
> When master crashs, HBaseAdmin will leaks ZooKeeper connections
> I think we should close the zk connetion when throw MasterNotRunningException
>  public HBaseAdmin(Configuration c)
>   throws MasterNotRunningException, ZooKeeperConnectionException {
>     this.conf = HBaseConfiguration.create(c);
>     this.connection = HConnectionManager.getConnection(this.conf);
>     this.pause = this.conf.getLong("hbase.client.pause", 1000);
>     this.numRetries = this.conf.getInt("hbase.client.retries.number", 10);
>     this.retryLongerMultiplier = this.conf.getInt("hbase.client.retries.longer.multiplier", 10);
>     //we should add this code and close the zk connection
>     try{
>       this.connection.getMaster();
>     }catch(MasterNotRunningException e){
>       HConnectionManager.deleteConnection(conf, false);
>       throw e;      
>     }
>   }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-4773) HBaseAdmin leaks ZooKeeper connections

Posted by "xufeng (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-4773?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13157024#comment-13157024 ] 

xufeng commented on HBASE-4773:
-------------------------------

yes, I have tested it in my cluster.

Here is my client test code:
{noformat}
.....
  static void initHBase() throws ZooKeeperConnectionException
  {
    HBaseAdmin hbaseAdmin = null;
    Configuration config = HBaseConfiguration.create();
    config.set("hbase.zookeeper.quorum", "158.1.130.31,158.1.130.32,158.1.130.33");
    config.set("hbase.zookeeper.property.clientPort", "2181");
    
    try {
      hbaseAdmin = new HBaseAdmin(config);
      System.out.println("init sucess!");
    } catch (MasterNotRunningException e) {
      e.printStackTrace();
      initHBase();
      
    } catch (ZooKeeperConnectionException e) {
      e.printStackTrace();
      initHBase();
    }
  }
}
.....
{noformat}

In my cluster I did not start HBase process.

Run test,result of the lsof commondline is:
{noformat}
java      16735       root   72w      REG              253,3   890569     524379 /opt/xf/hadoop.log
java      16735       root   73w      REG              253,3   274338     524376 /opt/xf/HA_hadoop.log
java      16735       root   74r     FIFO                0,8      0t0  110645029 pipe
java      16735       root   75w     FIFO                0,8      0t0  110645029 pipe
java      16735       root   76u     0000                0,9        0         21 anon_inode
java      16735       root   77u     IPv6          110645030      0t0        TCP C3S31:35186->C3S33:eforward (ESTABLISHED)
java      16735       root   78u     unix 0xffff8800cba90380      0t0  110645035 socket
java      16735       root   79u     sock                0,6      0t0  110645032 can't identify protocol
java      16735       root   80r     FIFO                0,8      0t0  110645037 pipe
java      16735       root   81w     FIFO                0,8      0t0  110645037 pipe
java      16735       root   82u     0000                0,9        0         21 anon_inode
java      16735       root   83u     IPv6          110645038      0t0        TCP C3S31:53727->C3S31:eforward (ESTABLISHED)
java      16735       root   84r     FIFO                0,8      0t0  110645043 pipe
java      16735       root   85w     FIFO                0,8      0t0  110645043 pipe
java      16735       root   86u     0000                0,9        0         21 anon_inode
java      16735       root   87u     IPv6          110645044      0t0        TCP C3S31:53728->C3S31:eforward (ESTABLISHED)
java      16735       root   88r     FIFO                0,8      0t0  110645047 pipe
java      16735       root   89w     FIFO                0,8      0t0  110645047 pipe
java      16735       root   90u     0000                0,9        0         21 anon_inode
java      16735       root   91u     IPv6          110645048      0t0        TCP C3S31:47183->C3S32:eforward (ESTABLISHED)
java      16735       root   92r     FIFO                0,8      0t0  110645050 pipe
java      16735       root   93w     FIFO                0,8      0t0  110645050 pipe
java      16735       root   94u     0000                0,9        0         21 anon_inode
java      16735       root   95u     IPv6          110645051      0t0        TCP C3S31:53730->C3S31:eforward (ESTABLISHED)
java      16735       root   96r     FIFO                0,8      0t0  110645135 pipe
java      16735       root   97w     FIFO                0,8      0t0  110645135 pipe
java      16735       root   98u     0000                0,9        0         21 anon_inode
java      16735       root   99u     IPv6          110645136      0t0        TCP C3S31:49799->C3S31:eforward (ESTABLISHED)
java      16735       root  100r     FIFO                0,8      0t0  110645143 pipe
java      16735       root  101w     FIFO                0,8      0t0  110645143 pipe
java      16735       root  102u     0000                0,9        0         21 anon_inode
java      16735       root  103u     IPv6          110645144      0t0        TCP C3S31:38931->C3S32:eforward (ESTABLISHED)
java      16735       root  104r     FIFO                0,8      0t0  110645148 pipe
java      16735       root  105w     FIFO                0,8      0t0  110645148 pipe
java      16735       root  106u     0000                0,9        0         21 anon_inode
java      16735       root  107u     IPv6          110645149      0t0        TCP C3S31:59939->C3S33:eforward (ESTABLISHED)
java      16735       root  108r     FIFO                0,8      0t0  110645507 pipe
java      16735       root  109w     FIFO                0,8      0t0  110645507 pipe
java      16735       root  110u     0000                0,9        0         21 anon_inode
java      16735       root  111u     IPv6          110645508      0t0        TCP C3S31:59940->C3S33:eforward (ESTABLISHED)
{noformat}

The [eforward] is port of zookeeper. 

It made the connection leak because did not delete connection between client and zookeeper when MasterNotRunningException happened.

And I also tested my patch,the result of it is :
{noformat}
java      16652       root   71r      REG              253,3   936397     524302 /opt/xf/lib/guava-r06.jar
java      16652       root   72w      REG              253,3   786418     524379 /opt/xf/hadoop.log
java      16652       root   73w      REG              253,3   262352     524376 /opt/xf/HA_hadoop.log
java      16652       root   74r     FIFO                0,8      0t0  110644817 pipe
java      16652       root   75w     FIFO                0,8      0t0  110644817 pipe
java      16652       root   76u     0000                0,9        0         21 anon_inode
java      16652       root   77u     IPv6          110644818      0t0        TCP C3S31:53993->C3S33:eforward (ESTABLISHED)
java      16652       root   78u     unix 0xffff8800cbb1d9c0      0t0  110644491 socket
java      16652       root   79u     sock                0,6      0t0  110644488 can't identify protocol
{noformat}
                
> HBaseAdmin leaks ZooKeeper connections
> --------------------------------------
>
>                 Key: HBASE-4773
>                 URL: https://issues.apache.org/jira/browse/HBASE-4773
>             Project: HBase
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 0.90.4
>            Reporter: gaojinchao
>            Priority: Critical
>             Fix For: 0.90.5
>
>         Attachments: 4773.patch
>
>
> When master crashs, HBaseAdmin will leaks ZooKeeper connections
> I think we should close the zk connetion when throw MasterNotRunningException
>  public HBaseAdmin(Configuration c)
>   throws MasterNotRunningException, ZooKeeperConnectionException {
>     this.conf = HBaseConfiguration.create(c);
>     this.connection = HConnectionManager.getConnection(this.conf);
>     this.pause = this.conf.getLong("hbase.client.pause", 1000);
>     this.numRetries = this.conf.getInt("hbase.client.retries.number", 10);
>     this.retryLongerMultiplier = this.conf.getInt("hbase.client.retries.longer.multiplier", 10);
>     //we should add this code and close the zk connection
>     try{
>       this.connection.getMaster();
>     }catch(MasterNotRunningException e){
>       HConnectionManager.deleteConnection(conf, false);
>       throw e;      
>     }
>   }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-4773) HBaseAdmin may leak ZooKeeper connections

Posted by "Hudson (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-4773?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13158994#comment-13158994 ] 

Hudson commented on HBASE-4773:
-------------------------------

Integrated in HBase-0.92-security #22 (See [https://builds.apache.org/job/HBase-0.92-security/22/])
    HBASE-4773  HBaseAdmin may leak ZooKeeper connections (Xufeng)

tedyu : 
Files : 
* /hbase/branches/0.92/CHANGES.txt
* /hbase/branches/0.92/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java

                
> HBaseAdmin may leak ZooKeeper connections
> -----------------------------------------
>
>                 Key: HBASE-4773
>                 URL: https://issues.apache.org/jira/browse/HBASE-4773
>             Project: HBase
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 0.90.4
>            Reporter: gaojinchao
>            Assignee: xufeng
>            Priority: Critical
>             Fix For: 0.90.5
>
>         Attachments: 4773.patch, branches_4773.patch, trunk_4773_patch.patch
>
>
> When master crashs, HBaseAdmin will leaks ZooKeeper connections
> I think we should close the zk connetion when throw MasterNotRunningException
>  public HBaseAdmin(Configuration c)
>   throws MasterNotRunningException, ZooKeeperConnectionException {
>     this.conf = HBaseConfiguration.create(c);
>     this.connection = HConnectionManager.getConnection(this.conf);
>     this.pause = this.conf.getLong("hbase.client.pause", 1000);
>     this.numRetries = this.conf.getInt("hbase.client.retries.number", 10);
>     this.retryLongerMultiplier = this.conf.getInt("hbase.client.retries.longer.multiplier", 10);
>     //we should add this code and close the zk connection
>     try{
>       this.connection.getMaster();
>     }catch(MasterNotRunningException e){
>       HConnectionManager.deleteConnection(conf, false);
>       throw e;      
>     }
>   }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-4773) HBaseAdmin may leak ZooKeeper connections

Posted by "Hudson (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-4773?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13158581#comment-13158581 ] 

Hudson commented on HBASE-4773:
-------------------------------

Integrated in HBase-TRUNK #2492 (See [https://builds.apache.org/job/HBase-TRUNK/2492/])
    HBASE-4773  HBaseAdmin may leak ZooKeeper connections (Xufeng)

tedyu : 
Files : 
* /hbase/trunk/CHANGES.txt
* /hbase/trunk/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java

                
> HBaseAdmin may leak ZooKeeper connections
> -----------------------------------------
>
>                 Key: HBASE-4773
>                 URL: https://issues.apache.org/jira/browse/HBASE-4773
>             Project: HBase
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 0.90.4
>            Reporter: gaojinchao
>            Assignee: xufeng
>            Priority: Critical
>             Fix For: 0.90.5
>
>         Attachments: 4773.patch, branches_4773.patch, trunk_4773_patch.patch
>
>
> When master crashs, HBaseAdmin will leaks ZooKeeper connections
> I think we should close the zk connetion when throw MasterNotRunningException
>  public HBaseAdmin(Configuration c)
>   throws MasterNotRunningException, ZooKeeperConnectionException {
>     this.conf = HBaseConfiguration.create(c);
>     this.connection = HConnectionManager.getConnection(this.conf);
>     this.pause = this.conf.getLong("hbase.client.pause", 1000);
>     this.numRetries = this.conf.getInt("hbase.client.retries.number", 10);
>     this.retryLongerMultiplier = this.conf.getInt("hbase.client.retries.longer.multiplier", 10);
>     //we should add this code and close the zk connection
>     try{
>       this.connection.getMaster();
>     }catch(MasterNotRunningException e){
>       HConnectionManager.deleteConnection(conf, false);
>       throw e;      
>     }
>   }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-4773) HBaseAdmin leaks ZooKeeper connections

Posted by "xufeng (Updated) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-4773?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

xufeng updated HBASE-4773:
--------------------------

    Attachment: trunk_4773_patch.patch
                branches_4773.patch

submit the branches and trunk patch.
unit test:ok.
test in cluster:ok
                
> HBaseAdmin leaks ZooKeeper connections
> --------------------------------------
>
>                 Key: HBASE-4773
>                 URL: https://issues.apache.org/jira/browse/HBASE-4773
>             Project: HBase
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 0.90.4
>            Reporter: gaojinchao
>            Priority: Critical
>             Fix For: 0.90.5
>
>         Attachments: 4773.patch, branches_4773.patch, trunk_4773_patch.patch
>
>
> When master crashs, HBaseAdmin will leaks ZooKeeper connections
> I think we should close the zk connetion when throw MasterNotRunningException
>  public HBaseAdmin(Configuration c)
>   throws MasterNotRunningException, ZooKeeperConnectionException {
>     this.conf = HBaseConfiguration.create(c);
>     this.connection = HConnectionManager.getConnection(this.conf);
>     this.pause = this.conf.getLong("hbase.client.pause", 1000);
>     this.numRetries = this.conf.getInt("hbase.client.retries.number", 10);
>     this.retryLongerMultiplier = this.conf.getInt("hbase.client.retries.longer.multiplier", 10);
>     //we should add this code and close the zk connection
>     try{
>       this.connection.getMaster();
>     }catch(MasterNotRunningException e){
>       HConnectionManager.deleteConnection(conf, false);
>       throw e;      
>     }
>   }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-4773) HBaseAdmin leaks ZooKeeper connections

Posted by "ramkrishna.s.vasudevan (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-4773?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13155319#comment-13155319 ] 

ramkrishna.s.vasudevan commented on HBASE-4773:
-----------------------------------------------

@Xufeng
Have you tested the patch in real cluster ?

                
> HBaseAdmin leaks ZooKeeper connections
> --------------------------------------
>
>                 Key: HBASE-4773
>                 URL: https://issues.apache.org/jira/browse/HBASE-4773
>             Project: HBase
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 0.90.4
>            Reporter: gaojinchao
>            Priority: Critical
>             Fix For: 0.90.5
>
>         Attachments: 4773.patch
>
>
> When master crashs, HBaseAdmin will leaks ZooKeeper connections
> I think we should close the zk connetion when throw MasterNotRunningException
>  public HBaseAdmin(Configuration c)
>   throws MasterNotRunningException, ZooKeeperConnectionException {
>     this.conf = HBaseConfiguration.create(c);
>     this.connection = HConnectionManager.getConnection(this.conf);
>     this.pause = this.conf.getLong("hbase.client.pause", 1000);
>     this.numRetries = this.conf.getInt("hbase.client.retries.number", 10);
>     this.retryLongerMultiplier = this.conf.getInt("hbase.client.retries.longer.multiplier", 10);
>     //we should add this code and close the zk connection
>     try{
>       this.connection.getMaster();
>     }catch(MasterNotRunningException e){
>       HConnectionManager.deleteConnection(conf, false);
>       throw e;      
>     }
>   }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-4773) HBaseAdmin may leak ZooKeeper connections

Posted by "Ted Yu (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-4773?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13158569#comment-13158569 ] 

Ted Yu commented on HBASE-4773:
-------------------------------

Integrated to 0.90, 0.92 and TRUNK.

Thanks for the patch Xufeng.

Thanks for the review Jinchao and Ramkrishna.
                
> HBaseAdmin may leak ZooKeeper connections
> -----------------------------------------
>
>                 Key: HBASE-4773
>                 URL: https://issues.apache.org/jira/browse/HBASE-4773
>             Project: HBase
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 0.90.4
>            Reporter: gaojinchao
>            Priority: Critical
>             Fix For: 0.90.5
>
>         Attachments: 4773.patch, branches_4773.patch, trunk_4773_patch.patch
>
>
> When master crashs, HBaseAdmin will leaks ZooKeeper connections
> I think we should close the zk connetion when throw MasterNotRunningException
>  public HBaseAdmin(Configuration c)
>   throws MasterNotRunningException, ZooKeeperConnectionException {
>     this.conf = HBaseConfiguration.create(c);
>     this.connection = HConnectionManager.getConnection(this.conf);
>     this.pause = this.conf.getLong("hbase.client.pause", 1000);
>     this.numRetries = this.conf.getInt("hbase.client.retries.number", 10);
>     this.retryLongerMultiplier = this.conf.getInt("hbase.client.retries.longer.multiplier", 10);
>     //we should add this code and close the zk connection
>     try{
>       this.connection.getMaster();
>     }catch(MasterNotRunningException e){
>       HConnectionManager.deleteConnection(conf, false);
>       throw e;      
>     }
>   }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-4773) HBaseAdmin leaks ZooKeeper connections

Posted by "Hadoop QA (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-4773?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13158406#comment-13158406 ] 

Hadoop QA commented on HBASE-4773:
----------------------------------

-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12505296/trunk_4773_patch.patch
  against trunk revision .

    +1 @author.  The patch does not contain any @author tags.

    -1 tests included.  The patch doesn't appear to include any new or modified tests.
                        Please justify why no new tests are needed for this patch.
                        Also please list what manual steps were performed to verify this patch.

    -1 javadoc.  The javadoc tool appears to have generated -162 warning messages.

    +1 javac.  The applied patch does not increase the total number of javac compiler warnings.

    -1 findbugs.  The patch appears to introduce 67 new Findbugs (version 1.3.9) warnings.

    +1 release audit.  The applied patch does not increase the total number of release audit warnings.

     -1 core tests.  The patch failed these unit tests:
                       org.apache.hadoop.hbase.TestFullLogReconstruction
                  org.apache.hadoop.hbase.mapreduce.TestImportTsv
                  org.apache.hadoop.hbase.mapreduce.TestTableMapReduce

Test results: https://builds.apache.org/job/PreCommit-HBASE-Build/392//testReport/
Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/392//artifact/trunk/patchprocess/newPatchFindbugsWarnings.html
Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/392//console

This message is automatically generated.
                
> HBaseAdmin leaks ZooKeeper connections
> --------------------------------------
>
>                 Key: HBASE-4773
>                 URL: https://issues.apache.org/jira/browse/HBASE-4773
>             Project: HBase
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 0.90.4
>            Reporter: gaojinchao
>            Priority: Critical
>             Fix For: 0.90.5
>
>         Attachments: 4773.patch, branches_4773.patch, trunk_4773_patch.patch
>
>
> When master crashs, HBaseAdmin will leaks ZooKeeper connections
> I think we should close the zk connetion when throw MasterNotRunningException
>  public HBaseAdmin(Configuration c)
>   throws MasterNotRunningException, ZooKeeperConnectionException {
>     this.conf = HBaseConfiguration.create(c);
>     this.connection = HConnectionManager.getConnection(this.conf);
>     this.pause = this.conf.getLong("hbase.client.pause", 1000);
>     this.numRetries = this.conf.getInt("hbase.client.retries.number", 10);
>     this.retryLongerMultiplier = this.conf.getInt("hbase.client.retries.longer.multiplier", 10);
>     //we should add this code and close the zk connection
>     try{
>       this.connection.getMaster();
>     }catch(MasterNotRunningException e){
>       HConnectionManager.deleteConnection(conf, false);
>       throw e;      
>     }
>   }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-4773) HBaseAdmin leaks ZooKeeper connections

Posted by "Ted Yu (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-4773?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13158249#comment-13158249 ] 

Ted Yu commented on HBASE-4773:
-------------------------------

@Xufeng:
HadoopQA isn't functional at the moment.
Please clarify whether you have run patch for TRUNK through unit test suite.
                
> HBaseAdmin leaks ZooKeeper connections
> --------------------------------------
>
>                 Key: HBASE-4773
>                 URL: https://issues.apache.org/jira/browse/HBASE-4773
>             Project: HBase
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 0.90.4
>            Reporter: gaojinchao
>            Priority: Critical
>             Fix For: 0.90.5
>
>         Attachments: 4773.patch, branches_4773.patch, trunk_4773_patch.patch
>
>
> When master crashs, HBaseAdmin will leaks ZooKeeper connections
> I think we should close the zk connetion when throw MasterNotRunningException
>  public HBaseAdmin(Configuration c)
>   throws MasterNotRunningException, ZooKeeperConnectionException {
>     this.conf = HBaseConfiguration.create(c);
>     this.connection = HConnectionManager.getConnection(this.conf);
>     this.pause = this.conf.getLong("hbase.client.pause", 1000);
>     this.numRetries = this.conf.getInt("hbase.client.retries.number", 10);
>     this.retryLongerMultiplier = this.conf.getInt("hbase.client.retries.longer.multiplier", 10);
>     //we should add this code and close the zk connection
>     try{
>       this.connection.getMaster();
>     }catch(MasterNotRunningException e){
>       HConnectionManager.deleteConnection(conf, false);
>       throw e;      
>     }
>   }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-4773) HBaseAdmin may leak ZooKeeper connections

Posted by "Hudson (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-4773?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13159130#comment-13159130 ] 

Hudson commented on HBASE-4773:
-------------------------------

Integrated in HBase-TRUNK-security #13 (See [https://builds.apache.org/job/HBase-TRUNK-security/13/])
    HBASE-4773  HBaseAdmin may leak ZooKeeper connections (Xufeng)

tedyu : 
Files : 
* /hbase/trunk/CHANGES.txt
* /hbase/trunk/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java

                
> HBaseAdmin may leak ZooKeeper connections
> -----------------------------------------
>
>                 Key: HBASE-4773
>                 URL: https://issues.apache.org/jira/browse/HBASE-4773
>             Project: HBase
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 0.90.4
>            Reporter: gaojinchao
>            Assignee: xufeng
>            Priority: Critical
>             Fix For: 0.90.5
>
>         Attachments: 4773.patch, branches_4773.patch, trunk_4773_patch.patch
>
>
> When master crashs, HBaseAdmin will leaks ZooKeeper connections
> I think we should close the zk connetion when throw MasterNotRunningException
>  public HBaseAdmin(Configuration c)
>   throws MasterNotRunningException, ZooKeeperConnectionException {
>     this.conf = HBaseConfiguration.create(c);
>     this.connection = HConnectionManager.getConnection(this.conf);
>     this.pause = this.conf.getLong("hbase.client.pause", 1000);
>     this.numRetries = this.conf.getInt("hbase.client.retries.number", 10);
>     this.retryLongerMultiplier = this.conf.getInt("hbase.client.retries.longer.multiplier", 10);
>     //we should add this code and close the zk connection
>     try{
>       this.connection.getMaster();
>     }catch(MasterNotRunningException e){
>       HConnectionManager.deleteConnection(conf, false);
>       throw e;      
>     }
>   }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-4773) HBaseAdmin may leak ZooKeeper connections

Posted by "ramkrishna.s.vasudevan (Updated) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-4773?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

ramkrishna.s.vasudevan updated HBASE-4773:
------------------------------------------

       Resolution: Fixed
    Fix Version/s: 0.92.0
           Status: Resolved  (was: Patch Available)

Committed sometime back.
                
> HBaseAdmin may leak ZooKeeper connections
> -----------------------------------------
>
>                 Key: HBASE-4773
>                 URL: https://issues.apache.org/jira/browse/HBASE-4773
>             Project: HBase
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 0.90.4
>            Reporter: gaojinchao
>            Assignee: xufeng
>            Priority: Critical
>             Fix For: 0.92.0, 0.90.6
>
>         Attachments: 4773.patch, branches_4773.patch, trunk_4773_patch.patch
>
>
> When master crashs, HBaseAdmin will leaks ZooKeeper connections
> I think we should close the zk connetion when throw MasterNotRunningException
>  public HBaseAdmin(Configuration c)
>   throws MasterNotRunningException, ZooKeeperConnectionException {
>     this.conf = HBaseConfiguration.create(c);
>     this.connection = HConnectionManager.getConnection(this.conf);
>     this.pause = this.conf.getLong("hbase.client.pause", 1000);
>     this.numRetries = this.conf.getInt("hbase.client.retries.number", 10);
>     this.retryLongerMultiplier = this.conf.getInt("hbase.client.retries.longer.multiplier", 10);
>     //we should add this code and close the zk connection
>     try{
>       this.connection.getMaster();
>     }catch(MasterNotRunningException e){
>       HConnectionManager.deleteConnection(conf, false);
>       throw e;      
>     }
>   }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HBASE-4773) HBaseAdmin may leak ZooKeeper connections

Posted by "Ted Yu (Updated) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-4773?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ted Yu updated HBASE-4773:
--------------------------

    Summary: HBaseAdmin may leak ZooKeeper connections  (was: HBaseAdmin leaks ZooKeeper connections)
    
> HBaseAdmin may leak ZooKeeper connections
> -----------------------------------------
>
>                 Key: HBASE-4773
>                 URL: https://issues.apache.org/jira/browse/HBASE-4773
>             Project: HBase
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 0.90.4
>            Reporter: gaojinchao
>            Priority: Critical
>             Fix For: 0.90.5
>
>         Attachments: 4773.patch, branches_4773.patch, trunk_4773_patch.patch
>
>
> When master crashs, HBaseAdmin will leaks ZooKeeper connections
> I think we should close the zk connetion when throw MasterNotRunningException
>  public HBaseAdmin(Configuration c)
>   throws MasterNotRunningException, ZooKeeperConnectionException {
>     this.conf = HBaseConfiguration.create(c);
>     this.connection = HConnectionManager.getConnection(this.conf);
>     this.pause = this.conf.getLong("hbase.client.pause", 1000);
>     this.numRetries = this.conf.getInt("hbase.client.retries.number", 10);
>     this.retryLongerMultiplier = this.conf.getInt("hbase.client.retries.longer.multiplier", 10);
>     //we should add this code and close the zk connection
>     try{
>       this.connection.getMaster();
>     }catch(MasterNotRunningException e){
>       HConnectionManager.deleteConnection(conf, false);
>       throw e;      
>     }
>   }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-4773) HBaseAdmin leaks ZooKeeper connections

Posted by "Ted Yu (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-4773?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13157198#comment-13157198 ] 

Ted Yu commented on HBASE-4773:
-------------------------------

In TRUNK, we retry connecting to master several times:
{code}
      } catch (MasterNotRunningException mnre) {
        HConnectionManager.deleteStaleConnection(this.connection);
        this.connection = HConnectionManager.getConnection(this.conf);
{code}

@Xufeng:
Can you implement similar retry loop for 0.90 ?

Thanks
                
> HBaseAdmin leaks ZooKeeper connections
> --------------------------------------
>
>                 Key: HBASE-4773
>                 URL: https://issues.apache.org/jira/browse/HBASE-4773
>             Project: HBase
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 0.90.4
>            Reporter: gaojinchao
>            Priority: Critical
>             Fix For: 0.90.5
>
>         Attachments: 4773.patch
>
>
> When master crashs, HBaseAdmin will leaks ZooKeeper connections
> I think we should close the zk connetion when throw MasterNotRunningException
>  public HBaseAdmin(Configuration c)
>   throws MasterNotRunningException, ZooKeeperConnectionException {
>     this.conf = HBaseConfiguration.create(c);
>     this.connection = HConnectionManager.getConnection(this.conf);
>     this.pause = this.conf.getLong("hbase.client.pause", 1000);
>     this.numRetries = this.conf.getInt("hbase.client.retries.number", 10);
>     this.retryLongerMultiplier = this.conf.getInt("hbase.client.retries.longer.multiplier", 10);
>     //we should add this code and close the zk connection
>     try{
>       this.connection.getMaster();
>     }catch(MasterNotRunningException e){
>       HConnectionManager.deleteConnection(conf, false);
>       throw e;      
>     }
>   }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-4773) HBaseAdmin leaks ZooKeeper connections

Posted by "ramkrishna.s.vasudevan (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-4773?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13158341#comment-13158341 ] 

ramkrishna.s.vasudevan commented on HBASE-4773:
-----------------------------------------------

+1
                
> HBaseAdmin leaks ZooKeeper connections
> --------------------------------------
>
>                 Key: HBASE-4773
>                 URL: https://issues.apache.org/jira/browse/HBASE-4773
>             Project: HBase
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 0.90.4
>            Reporter: gaojinchao
>            Priority: Critical
>             Fix For: 0.90.5
>
>         Attachments: 4773.patch, branches_4773.patch, trunk_4773_patch.patch
>
>
> When master crashs, HBaseAdmin will leaks ZooKeeper connections
> I think we should close the zk connetion when throw MasterNotRunningException
>  public HBaseAdmin(Configuration c)
>   throws MasterNotRunningException, ZooKeeperConnectionException {
>     this.conf = HBaseConfiguration.create(c);
>     this.connection = HConnectionManager.getConnection(this.conf);
>     this.pause = this.conf.getLong("hbase.client.pause", 1000);
>     this.numRetries = this.conf.getInt("hbase.client.retries.number", 10);
>     this.retryLongerMultiplier = this.conf.getInt("hbase.client.retries.longer.multiplier", 10);
>     //we should add this code and close the zk connection
>     try{
>       this.connection.getMaster();
>     }catch(MasterNotRunningException e){
>       HConnectionManager.deleteConnection(conf, false);
>       throw e;      
>     }
>   }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-4773) HBaseAdmin leaks ZooKeeper connections

Posted by "xufeng (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-4773?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13158293#comment-13158293 ] 

xufeng commented on HBASE-4773:
-------------------------------

@Ted 
yes,I have run patch for TRUNK through unit test suite in my env.
                
> HBaseAdmin leaks ZooKeeper connections
> --------------------------------------
>
>                 Key: HBASE-4773
>                 URL: https://issues.apache.org/jira/browse/HBASE-4773
>             Project: HBase
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 0.90.4
>            Reporter: gaojinchao
>            Priority: Critical
>             Fix For: 0.90.5
>
>         Attachments: 4773.patch, branches_4773.patch, trunk_4773_patch.patch
>
>
> When master crashs, HBaseAdmin will leaks ZooKeeper connections
> I think we should close the zk connetion when throw MasterNotRunningException
>  public HBaseAdmin(Configuration c)
>   throws MasterNotRunningException, ZooKeeperConnectionException {
>     this.conf = HBaseConfiguration.create(c);
>     this.connection = HConnectionManager.getConnection(this.conf);
>     this.pause = this.conf.getLong("hbase.client.pause", 1000);
>     this.numRetries = this.conf.getInt("hbase.client.retries.number", 10);
>     this.retryLongerMultiplier = this.conf.getInt("hbase.client.retries.longer.multiplier", 10);
>     //we should add this code and close the zk connection
>     try{
>       this.connection.getMaster();
>     }catch(MasterNotRunningException e){
>       HConnectionManager.deleteConnection(conf, false);
>       throw e;      
>     }
>   }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-4773) HBaseAdmin may leak ZooKeeper connections

Posted by "xufeng (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-4773?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13159211#comment-13159211 ] 

xufeng commented on HBASE-4773:
-------------------------------

Thanks everyone.
It is a milestone to me.
                
> HBaseAdmin may leak ZooKeeper connections
> -----------------------------------------
>
>                 Key: HBASE-4773
>                 URL: https://issues.apache.org/jira/browse/HBASE-4773
>             Project: HBase
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 0.90.4
>            Reporter: gaojinchao
>            Assignee: xufeng
>            Priority: Critical
>             Fix For: 0.90.5
>
>         Attachments: 4773.patch, branches_4773.patch, trunk_4773_patch.patch
>
>
> When master crashs, HBaseAdmin will leaks ZooKeeper connections
> I think we should close the zk connetion when throw MasterNotRunningException
>  public HBaseAdmin(Configuration c)
>   throws MasterNotRunningException, ZooKeeperConnectionException {
>     this.conf = HBaseConfiguration.create(c);
>     this.connection = HConnectionManager.getConnection(this.conf);
>     this.pause = this.conf.getLong("hbase.client.pause", 1000);
>     this.numRetries = this.conf.getInt("hbase.client.retries.number", 10);
>     this.retryLongerMultiplier = this.conf.getInt("hbase.client.retries.longer.multiplier", 10);
>     //we should add this code and close the zk connection
>     try{
>       this.connection.getMaster();
>     }catch(MasterNotRunningException e){
>       HConnectionManager.deleteConnection(conf, false);
>       throw e;      
>     }
>   }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-4773) HBaseAdmin leaks ZooKeeper connections

Posted by "Ted Yu (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-4773?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13157069#comment-13157069 ] 

Ted Yu commented on HBASE-4773:
-------------------------------

+1 on patch. 

Can you make a patch for trunk ?
                
> HBaseAdmin leaks ZooKeeper connections
> --------------------------------------
>
>                 Key: HBASE-4773
>                 URL: https://issues.apache.org/jira/browse/HBASE-4773
>             Project: HBase
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 0.90.4
>            Reporter: gaojinchao
>            Priority: Critical
>             Fix For: 0.90.5
>
>         Attachments: 4773.patch
>
>
> When master crashs, HBaseAdmin will leaks ZooKeeper connections
> I think we should close the zk connetion when throw MasterNotRunningException
>  public HBaseAdmin(Configuration c)
>   throws MasterNotRunningException, ZooKeeperConnectionException {
>     this.conf = HBaseConfiguration.create(c);
>     this.connection = HConnectionManager.getConnection(this.conf);
>     this.pause = this.conf.getLong("hbase.client.pause", 1000);
>     this.numRetries = this.conf.getInt("hbase.client.retries.number", 10);
>     this.retryLongerMultiplier = this.conf.getInt("hbase.client.retries.longer.multiplier", 10);
>     //we should add this code and close the zk connection
>     try{
>       this.connection.getMaster();
>     }catch(MasterNotRunningException e){
>       HConnectionManager.deleteConnection(conf, false);
>       throw e;      
>     }
>   }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HBASE-4773) HBaseAdmin leaks ZooKeeper connections

Posted by "gaojinchao (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-4773?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13157352#comment-13157352 ] 

gaojinchao commented on HBASE-4773:
-----------------------------------

In TRUNK, before throwing exception, we should call deleteStaleConnection to clean the dirty data

                
> HBaseAdmin leaks ZooKeeper connections
> --------------------------------------
>
>                 Key: HBASE-4773
>                 URL: https://issues.apache.org/jira/browse/HBASE-4773
>             Project: HBase
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 0.90.4
>            Reporter: gaojinchao
>            Priority: Critical
>             Fix For: 0.90.5
>
>         Attachments: 4773.patch
>
>
> When master crashs, HBaseAdmin will leaks ZooKeeper connections
> I think we should close the zk connetion when throw MasterNotRunningException
>  public HBaseAdmin(Configuration c)
>   throws MasterNotRunningException, ZooKeeperConnectionException {
>     this.conf = HBaseConfiguration.create(c);
>     this.connection = HConnectionManager.getConnection(this.conf);
>     this.pause = this.conf.getLong("hbase.client.pause", 1000);
>     this.numRetries = this.conf.getInt("hbase.client.retries.number", 10);
>     this.retryLongerMultiplier = this.conf.getInt("hbase.client.retries.longer.multiplier", 10);
>     //we should add this code and close the zk connection
>     try{
>       this.connection.getMaster();
>     }catch(MasterNotRunningException e){
>       HConnectionManager.deleteConnection(conf, false);
>       throw e;      
>     }
>   }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Assigned] (HBASE-4773) HBaseAdmin may leak ZooKeeper connections

Posted by "Ted Yu (Assigned) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HBASE-4773?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ted Yu reassigned HBASE-4773:
-----------------------------

    Assignee: xufeng
    
> HBaseAdmin may leak ZooKeeper connections
> -----------------------------------------
>
>                 Key: HBASE-4773
>                 URL: https://issues.apache.org/jira/browse/HBASE-4773
>             Project: HBase
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 0.90.4
>            Reporter: gaojinchao
>            Assignee: xufeng
>            Priority: Critical
>             Fix For: 0.90.5
>
>         Attachments: 4773.patch, branches_4773.patch, trunk_4773_patch.patch
>
>
> When master crashs, HBaseAdmin will leaks ZooKeeper connections
> I think we should close the zk connetion when throw MasterNotRunningException
>  public HBaseAdmin(Configuration c)
>   throws MasterNotRunningException, ZooKeeperConnectionException {
>     this.conf = HBaseConfiguration.create(c);
>     this.connection = HConnectionManager.getConnection(this.conf);
>     this.pause = this.conf.getLong("hbase.client.pause", 1000);
>     this.numRetries = this.conf.getInt("hbase.client.retries.number", 10);
>     this.retryLongerMultiplier = this.conf.getInt("hbase.client.retries.longer.multiplier", 10);
>     //we should add this code and close the zk connection
>     try{
>       this.connection.getMaster();
>     }catch(MasterNotRunningException e){
>       HConnectionManager.deleteConnection(conf, false);
>       throw e;      
>     }
>   }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira