[
https://issues.apache.org/jira/browse/HBASE-2828?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Nicolas Spiegelberg updated HBASE-2828:
---------------------------------------
Attachment: HBASE-2828.patch
I just kept this simple and used existing API calls to grab the zkWrapper, get
all RS nodes, and call size(). It would be nice if ZK provided a
getChildrenCount(), so it didn't have to list all the paths, but I assumed this
shouldn't be a significant performance concern, even for larger clusters.
Note that I also used this opportunity to clean up the class header
documentation for HTable & HBaseAdmin. Feel free to ignore those changes if
you have your own doc update.
> HTable unnecessarily coupled with HMaster
> -----------------------------------------
>
> Key: HBASE-2828
> URL: https://issues.apache.org/jira/browse/HBASE-2828
> Project: HBase
> Issue Type: Bug
> Components: client
> Affects Versions: 0.90.0
> Reporter: Nicolas Spiegelberg
> Fix For: 0.90.0
>
> Attachments: HBASE-2828.patch
>
>
> HTable constructor calls "getCurrentNrHRS()" to get the region server count
> for thread pool creation. This code calls HBaseAdmin.getClusterStatus()
> [aka: the HMaster] to get the server count. This information can be scraped
> from counting the ZooKeeper /hbase/rs/--- ZNodes. Need to remove unnecessary
> master queries when ZooKeeper can do the same job.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
