hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "stack (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-12835) HBASE-12422 changed new HTable(Configuration) to not use managed Connections anymore
Date Fri, 09 Jan 2015 23:09:34 GMT

    [ https://issues.apache.org/jira/browse/HBASE-12835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14272033#comment-14272033
] 

stack commented on HBASE-12835:
-------------------------------

This intended [~enis]?

253	    this.connection = (ClusterConnection) ConnectionFactory.createConnection(conf);		
253	    ConnectionManager.getConnectionInternal(conf);

Otherwise +1 if hadoopqa good.

> HBASE-12422 changed new HTable(Configuration) to not use managed Connections anymore
> ------------------------------------------------------------------------------------
>
>                 Key: HBASE-12835
>                 URL: https://issues.apache.org/jira/browse/HBASE-12835
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Enis Soztutar
>            Assignee: Enis Soztutar
>             Fix For: 1.0.0, 1.1.0
>
>         Attachments: hbase-12835_v1.patch
>
>
> While debugging something else, I've noticed that HBASE-12422 changed it so that new
HTable(Configuration, TableName) is not using managed connections anymore. 
> I thought the plan was to keep new HTable(Conf, TableName) to still use managed connections,
but allow the new APIs to have un-managed connections. With HBASE-12422, there is a big compatibility
change for used of the HTable interface that suddenly they will be launching connections per
HTable instance which is a behavioral change and expensive (with all the meta cache, zk threads,
thread pool, etc). I fear that this will catch unsuspecting users by surprise, that is why
I think we should revert back to the deprecated behavior for 1.0. For master we do not need
to make the change, but document it. 
> [~sduskis], [~saint.ack@gmail.com], [~ndimiduk] what do you guys think?



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message