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 22:33:35 GMT

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

stack commented on HBASE-12835:

I see your point [~enis] Over on the issue, thought concern was about closing and that had
been dealt with.  Lets be conservative and move this one back to managed connections (Might
explain the horrors [~jurmous] struggled with in TestHBaseFsck... where he did a bunch of
work to shut down number of running connections?)

> 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
> 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

View raw message