HBASE-2828 HTable unnecessarily coupled with HMaster -- Added comment that we do not want dependency on Master
git-svn-id: https://svn.apache.org/repos/asf/hbase/trunk@1030745 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
parent
dc5301c058
commit
f0e0f6c8a7
1
pom.xml
1
pom.xml
|
@ -261,6 +261,7 @@
|
||||||
<execution>
|
<execution>
|
||||||
<goals>
|
<goals>
|
||||||
<goal>generate-html</goal>
|
<goal>generate-html</goal>
|
||||||
|
<goal>generate-pdf</goal>
|
||||||
</goals>
|
</goals>
|
||||||
<phase>pre-site</phase>
|
<phase>pre-site</phase>
|
||||||
</execution>
|
</execution>
|
||||||
|
|
|
@ -197,13 +197,13 @@ public class HTable implements HTableInterface {
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* TODO Might want to change this to public, would be nice if the number
|
|
||||||
* of threads would automatically change when servers were added and removed
|
|
||||||
* @return the number of region servers that are currently running
|
* @return the number of region servers that are currently running
|
||||||
* @throws IOException if a remote or network exception occurs
|
* @throws IOException if a remote or network exception occurs
|
||||||
*/
|
*/
|
||||||
int getCurrentNrHRS() throws IOException {
|
public int getCurrentNrHRS() throws IOException {
|
||||||
try {
|
try {
|
||||||
|
// We go to zk rather than to master to get count of regions to avoid
|
||||||
|
// HTable having a Master dependency. See HBase-2828
|
||||||
return ZKUtil.getNumberOfChildren(this.connection.getZooKeeperWatcher(),
|
return ZKUtil.getNumberOfChildren(this.connection.getZooKeeperWatcher(),
|
||||||
this.connection.getZooKeeperWatcher().rsZNode);
|
this.connection.getZooKeeperWatcher().rsZNode);
|
||||||
} catch (KeeperException ke) {
|
} catch (KeeperException ke) {
|
||||||
|
|
Loading…
Reference in New Issue