You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by gi...@apache.org on 2017/12/17 15:17:46 UTC

[07/28] hbase-site git commit: Published site at .

http://git-wip-us.apache.org/repos/asf/hbase-site/blob/48bd3915/devapidocs/src-html/org/apache/hadoop/hbase/zookeeper/ZKWatcher.html
----------------------------------------------------------------------
diff --git a/devapidocs/src-html/org/apache/hadoop/hbase/zookeeper/ZKWatcher.html b/devapidocs/src-html/org/apache/hadoop/hbase/zookeeper/ZKWatcher.html
index 0c4f5b4..310c9ef 100644
--- a/devapidocs/src-html/org/apache/hadoop/hbase/zookeeper/ZKWatcher.html
+++ b/devapidocs/src-html/org/apache/hadoop/hbase/zookeeper/ZKWatcher.html
@@ -41,9 +41,9 @@
 <span class="sourceLineNo">033</span>import org.apache.hadoop.hbase.Abortable;<a name="line.33"></a>
 <span class="sourceLineNo">034</span>import org.apache.hadoop.hbase.AuthUtil;<a name="line.34"></a>
 <span class="sourceLineNo">035</span>import org.apache.hadoop.hbase.ZooKeeperConnectionException;<a name="line.35"></a>
-<span class="sourceLineNo">036</span>import org.apache.yetus.audience.InterfaceAudience;<a name="line.36"></a>
-<span class="sourceLineNo">037</span>import org.apache.hadoop.hbase.security.Superusers;<a name="line.37"></a>
-<span class="sourceLineNo">038</span>import org.apache.hadoop.security.UserGroupInformation;<a name="line.38"></a>
+<span class="sourceLineNo">036</span>import org.apache.hadoop.hbase.security.Superusers;<a name="line.36"></a>
+<span class="sourceLineNo">037</span>import org.apache.hadoop.security.UserGroupInformation;<a name="line.37"></a>
+<span class="sourceLineNo">038</span>import org.apache.yetus.audience.InterfaceAudience;<a name="line.38"></a>
 <span class="sourceLineNo">039</span>import org.apache.zookeeper.KeeperException;<a name="line.39"></a>
 <span class="sourceLineNo">040</span>import org.apache.zookeeper.WatchedEvent;<a name="line.40"></a>
 <span class="sourceLineNo">041</span>import org.apache.zookeeper.Watcher;<a name="line.41"></a>
@@ -93,294 +93,294 @@
 <span class="sourceLineNo">085</span>  // negotiation to complete<a name="line.85"></a>
 <span class="sourceLineNo">086</span>  public CountDownLatch saslLatch = new CountDownLatch(1);<a name="line.86"></a>
 <span class="sourceLineNo">087</span><a name="line.87"></a>
-<span class="sourceLineNo">088</span><a name="line.88"></a>
+<span class="sourceLineNo">088</span>  private final Configuration conf;<a name="line.88"></a>
 <span class="sourceLineNo">089</span><a name="line.89"></a>
-<span class="sourceLineNo">090</span>  private final Configuration conf;<a name="line.90"></a>
-<span class="sourceLineNo">091</span><a name="line.91"></a>
-<span class="sourceLineNo">092</span>  /* A pattern that matches a Kerberos name, borrowed from Hadoop's KerberosName */<a name="line.92"></a>
-<span class="sourceLineNo">093</span>  private static final Pattern NAME_PATTERN = Pattern.compile("([^/@]*)(/([^/@]*))?@([^/@]*)");<a name="line.93"></a>
-<span class="sourceLineNo">094</span><a name="line.94"></a>
-<span class="sourceLineNo">095</span>  /**<a name="line.95"></a>
-<span class="sourceLineNo">096</span>   * Instantiate a ZooKeeper connection and watcher.<a name="line.96"></a>
-<span class="sourceLineNo">097</span>   * @param identifier string that is passed to RecoverableZookeeper to be used as<a name="line.97"></a>
-<span class="sourceLineNo">098</span>   * identifier for this instance. Use null for default.<a name="line.98"></a>
-<span class="sourceLineNo">099</span>   * @throws IOException<a name="line.99"></a>
-<span class="sourceLineNo">100</span>   * @throws ZooKeeperConnectionException<a name="line.100"></a>
-<span class="sourceLineNo">101</span>   */<a name="line.101"></a>
-<span class="sourceLineNo">102</span>  public ZKWatcher(Configuration conf, String identifier,<a name="line.102"></a>
-<span class="sourceLineNo">103</span>                   Abortable abortable) throws ZooKeeperConnectionException, IOException {<a name="line.103"></a>
-<span class="sourceLineNo">104</span>    this(conf, identifier, abortable, false);<a name="line.104"></a>
-<span class="sourceLineNo">105</span>  }<a name="line.105"></a>
-<span class="sourceLineNo">106</span><a name="line.106"></a>
-<span class="sourceLineNo">107</span>  /**<a name="line.107"></a>
-<span class="sourceLineNo">108</span>   * Instantiate a ZooKeeper connection and watcher.<a name="line.108"></a>
-<span class="sourceLineNo">109</span>   * @param conf<a name="line.109"></a>
-<span class="sourceLineNo">110</span>   * @param identifier string that is passed to RecoverableZookeeper to be used as identifier for<a name="line.110"></a>
-<span class="sourceLineNo">111</span>   *          this instance. Use null for default.<a name="line.111"></a>
-<span class="sourceLineNo">112</span>   * @param abortable Can be null if there is on error there is no host to abort: e.g. client<a name="line.112"></a>
-<span class="sourceLineNo">113</span>   *          context.<a name="line.113"></a>
-<span class="sourceLineNo">114</span>   * @param canCreateBaseZNode<a name="line.114"></a>
-<span class="sourceLineNo">115</span>   * @throws IOException<a name="line.115"></a>
-<span class="sourceLineNo">116</span>   * @throws ZooKeeperConnectionException<a name="line.116"></a>
-<span class="sourceLineNo">117</span>   */<a name="line.117"></a>
-<span class="sourceLineNo">118</span>  public ZKWatcher(Configuration conf, String identifier,<a name="line.118"></a>
-<span class="sourceLineNo">119</span>                   Abortable abortable, boolean canCreateBaseZNode)<a name="line.119"></a>
-<span class="sourceLineNo">120</span>  throws IOException, ZooKeeperConnectionException {<a name="line.120"></a>
-<span class="sourceLineNo">121</span>    this.conf = conf;<a name="line.121"></a>
-<span class="sourceLineNo">122</span>    this.quorum = ZKConfig.getZKQuorumServersString(conf);<a name="line.122"></a>
-<span class="sourceLineNo">123</span>    this.prefix = identifier;<a name="line.123"></a>
-<span class="sourceLineNo">124</span>    // Identifier will get the sessionid appended later below down when we<a name="line.124"></a>
-<span class="sourceLineNo">125</span>    // handle the syncconnect event.<a name="line.125"></a>
-<span class="sourceLineNo">126</span>    this.identifier = identifier + "0x0";<a name="line.126"></a>
-<span class="sourceLineNo">127</span>    this.abortable = abortable;<a name="line.127"></a>
-<span class="sourceLineNo">128</span>    this.znodePaths = new ZNodePaths(conf);<a name="line.128"></a>
-<span class="sourceLineNo">129</span>    PendingWatcher pendingWatcher = new PendingWatcher();<a name="line.129"></a>
-<span class="sourceLineNo">130</span>    this.recoverableZooKeeper = ZKUtil.connect(conf, quorum, pendingWatcher, identifier);<a name="line.130"></a>
-<span class="sourceLineNo">131</span>    pendingWatcher.prepare(this);<a name="line.131"></a>
-<span class="sourceLineNo">132</span>    if (canCreateBaseZNode) {<a name="line.132"></a>
-<span class="sourceLineNo">133</span>      try {<a name="line.133"></a>
-<span class="sourceLineNo">134</span>        createBaseZNodes();<a name="line.134"></a>
-<span class="sourceLineNo">135</span>      } catch (ZooKeeperConnectionException zce) {<a name="line.135"></a>
-<span class="sourceLineNo">136</span>        try {<a name="line.136"></a>
-<span class="sourceLineNo">137</span>          this.recoverableZooKeeper.close();<a name="line.137"></a>
-<span class="sourceLineNo">138</span>        } catch (InterruptedException ie) {<a name="line.138"></a>
-<span class="sourceLineNo">139</span>          LOG.debug("Encountered InterruptedException when closing " + this.recoverableZooKeeper);<a name="line.139"></a>
-<span class="sourceLineNo">140</span>          Thread.currentThread().interrupt();<a name="line.140"></a>
-<span class="sourceLineNo">141</span>        }<a name="line.141"></a>
-<span class="sourceLineNo">142</span>        throw zce;<a name="line.142"></a>
-<span class="sourceLineNo">143</span>      }<a name="line.143"></a>
-<span class="sourceLineNo">144</span>    }<a name="line.144"></a>
-<span class="sourceLineNo">145</span>  }<a name="line.145"></a>
-<span class="sourceLineNo">146</span><a name="line.146"></a>
-<span class="sourceLineNo">147</span>  private void createBaseZNodes() throws ZooKeeperConnectionException {<a name="line.147"></a>
-<span class="sourceLineNo">148</span>    try {<a name="line.148"></a>
-<span class="sourceLineNo">149</span>      // Create all the necessary "directories" of znodes<a name="line.149"></a>
-<span class="sourceLineNo">150</span>      ZKUtil.createWithParents(this, znodePaths.baseZNode);<a name="line.150"></a>
-<span class="sourceLineNo">151</span>      ZKUtil.createAndFailSilent(this, znodePaths.rsZNode);<a name="line.151"></a>
-<span class="sourceLineNo">152</span>      ZKUtil.createAndFailSilent(this, znodePaths.drainingZNode);<a name="line.152"></a>
-<span class="sourceLineNo">153</span>      ZKUtil.createAndFailSilent(this, znodePaths.tableZNode);<a name="line.153"></a>
-<span class="sourceLineNo">154</span>      ZKUtil.createAndFailSilent(this, znodePaths.splitLogZNode);<a name="line.154"></a>
-<span class="sourceLineNo">155</span>      ZKUtil.createAndFailSilent(this, znodePaths.backupMasterAddressesZNode);<a name="line.155"></a>
-<span class="sourceLineNo">156</span>      ZKUtil.createAndFailSilent(this, znodePaths.tableLockZNode);<a name="line.156"></a>
-<span class="sourceLineNo">157</span>      ZKUtil.createAndFailSilent(this, znodePaths.masterMaintZNode);<a name="line.157"></a>
-<span class="sourceLineNo">158</span>    } catch (KeeperException e) {<a name="line.158"></a>
-<span class="sourceLineNo">159</span>      throw new ZooKeeperConnectionException(<a name="line.159"></a>
-<span class="sourceLineNo">160</span>          prefix("Unexpected KeeperException creating base node"), e);<a name="line.160"></a>
-<span class="sourceLineNo">161</span>    }<a name="line.161"></a>
-<span class="sourceLineNo">162</span>  }<a name="line.162"></a>
-<span class="sourceLineNo">163</span><a name="line.163"></a>
-<span class="sourceLineNo">164</span>  /** Returns whether the znode is supposed to be readable by the client<a name="line.164"></a>
-<span class="sourceLineNo">165</span>   * and DOES NOT contain sensitive information (world readable).*/<a name="line.165"></a>
-<span class="sourceLineNo">166</span>  public boolean isClientReadable(String node) {<a name="line.166"></a>
-<span class="sourceLineNo">167</span>    // Developer notice: These znodes are world readable. DO NOT add more znodes here UNLESS<a name="line.167"></a>
-<span class="sourceLineNo">168</span>    // all clients need to access this data to work. Using zk for sharing data to clients (other<a name="line.168"></a>
-<span class="sourceLineNo">169</span>    // than service lookup case is not a recommended design pattern.<a name="line.169"></a>
-<span class="sourceLineNo">170</span>    return<a name="line.170"></a>
-<span class="sourceLineNo">171</span>        node.equals(znodePaths.baseZNode) ||<a name="line.171"></a>
-<span class="sourceLineNo">172</span>        znodePaths.isAnyMetaReplicaZNode(node) ||<a name="line.172"></a>
-<span class="sourceLineNo">173</span>        node.equals(znodePaths.masterAddressZNode) ||<a name="line.173"></a>
-<span class="sourceLineNo">174</span>        node.equals(znodePaths.clusterIdZNode)||<a name="line.174"></a>
-<span class="sourceLineNo">175</span>        node.equals(znodePaths.rsZNode) ||<a name="line.175"></a>
-<span class="sourceLineNo">176</span>        // /hbase/table and /hbase/table/foo is allowed, /hbase/table-lock is not<a name="line.176"></a>
-<span class="sourceLineNo">177</span>        node.equals(znodePaths.tableZNode) ||<a name="line.177"></a>
-<span class="sourceLineNo">178</span>        node.startsWith(znodePaths.tableZNode + "/");<a name="line.178"></a>
-<span class="sourceLineNo">179</span>  }<a name="line.179"></a>
-<span class="sourceLineNo">180</span><a name="line.180"></a>
-<span class="sourceLineNo">181</span>  /**<a name="line.181"></a>
-<span class="sourceLineNo">182</span>   * On master start, we check the znode ACLs under the root directory and set the ACLs properly<a name="line.182"></a>
-<span class="sourceLineNo">183</span>   * if needed. If the cluster goes from an unsecure setup to a secure setup, this step is needed<a name="line.183"></a>
-<span class="sourceLineNo">184</span>   * so that the existing znodes created with open permissions are now changed with restrictive<a name="line.184"></a>
-<span class="sourceLineNo">185</span>   * perms.<a name="line.185"></a>
-<span class="sourceLineNo">186</span>   */<a name="line.186"></a>
-<span class="sourceLineNo">187</span>  public void checkAndSetZNodeAcls() {<a name="line.187"></a>
-<span class="sourceLineNo">188</span>    if (!ZKUtil.isSecureZooKeeper(getConfiguration())) {<a name="line.188"></a>
-<span class="sourceLineNo">189</span>      LOG.info("not a secure deployment, proceeding");<a name="line.189"></a>
-<span class="sourceLineNo">190</span>      return;<a name="line.190"></a>
-<span class="sourceLineNo">191</span>    }<a name="line.191"></a>
-<span class="sourceLineNo">192</span><a name="line.192"></a>
-<span class="sourceLineNo">193</span>    // Check the base znodes permission first. Only do the recursion if base znode's perms are not<a name="line.193"></a>
-<span class="sourceLineNo">194</span>    // correct.<a name="line.194"></a>
-<span class="sourceLineNo">195</span>    try {<a name="line.195"></a>
-<span class="sourceLineNo">196</span>      List&lt;ACL&gt; actualAcls = recoverableZooKeeper.getAcl(znodePaths.baseZNode, new Stat());<a name="line.196"></a>
-<span class="sourceLineNo">197</span><a name="line.197"></a>
-<span class="sourceLineNo">198</span>      if (!isBaseZnodeAclSetup(actualAcls)) {<a name="line.198"></a>
-<span class="sourceLineNo">199</span>        LOG.info("setting znode ACLs");<a name="line.199"></a>
-<span class="sourceLineNo">200</span>        setZnodeAclsRecursive(znodePaths.baseZNode);<a name="line.200"></a>
-<span class="sourceLineNo">201</span>      }<a name="line.201"></a>
-<span class="sourceLineNo">202</span>    } catch(KeeperException.NoNodeException nne) {<a name="line.202"></a>
-<span class="sourceLineNo">203</span>      return;<a name="line.203"></a>
-<span class="sourceLineNo">204</span>    } catch(InterruptedException ie) {<a name="line.204"></a>
-<span class="sourceLineNo">205</span>      interruptedExceptionNoThrow(ie, false);<a name="line.205"></a>
-<span class="sourceLineNo">206</span>    } catch (IOException|KeeperException e) {<a name="line.206"></a>
-<span class="sourceLineNo">207</span>      LOG.warn("Received exception while checking and setting zookeeper ACLs", e);<a name="line.207"></a>
-<span class="sourceLineNo">208</span>    }<a name="line.208"></a>
-<span class="sourceLineNo">209</span>  }<a name="line.209"></a>
-<span class="sourceLineNo">210</span><a name="line.210"></a>
-<span class="sourceLineNo">211</span>  /**<a name="line.211"></a>
-<span class="sourceLineNo">212</span>   * Set the znode perms recursively. This will do post-order recursion, so that baseZnode ACLs<a name="line.212"></a>
-<span class="sourceLineNo">213</span>   * will be set last in case the master fails in between.<a name="line.213"></a>
-<span class="sourceLineNo">214</span>   * @param znode<a name="line.214"></a>
-<span class="sourceLineNo">215</span>   */<a name="line.215"></a>
-<span class="sourceLineNo">216</span>  private void setZnodeAclsRecursive(String znode) throws KeeperException, InterruptedException {<a name="line.216"></a>
-<span class="sourceLineNo">217</span>    List&lt;String&gt; children = recoverableZooKeeper.getChildren(znode, false);<a name="line.217"></a>
-<span class="sourceLineNo">218</span><a name="line.218"></a>
-<span class="sourceLineNo">219</span>    for (String child : children) {<a name="line.219"></a>
-<span class="sourceLineNo">220</span>      setZnodeAclsRecursive(ZNodePaths.joinZNode(znode, child));<a name="line.220"></a>
-<span class="sourceLineNo">221</span>    }<a name="line.221"></a>
-<span class="sourceLineNo">222</span>    List&lt;ACL&gt; acls = ZKUtil.createACL(this, znode, true);<a name="line.222"></a>
-<span class="sourceLineNo">223</span>    LOG.info("Setting ACLs for znode:" + znode + " , acl:" + acls);<a name="line.223"></a>
-<span class="sourceLineNo">224</span>    recoverableZooKeeper.setAcl(znode, acls, -1);<a name="line.224"></a>
-<span class="sourceLineNo">225</span>  }<a name="line.225"></a>
-<span class="sourceLineNo">226</span><a name="line.226"></a>
-<span class="sourceLineNo">227</span>  /**<a name="line.227"></a>
-<span class="sourceLineNo">228</span>   * Checks whether the ACLs returned from the base znode (/hbase) is set for secure setup.<a name="line.228"></a>
-<span class="sourceLineNo">229</span>   * @param acls acls from zookeeper<a name="line.229"></a>
-<span class="sourceLineNo">230</span>   * @return whether ACLs are set for the base znode<a name="line.230"></a>
-<span class="sourceLineNo">231</span>   * @throws IOException<a name="line.231"></a>
-<span class="sourceLineNo">232</span>   */<a name="line.232"></a>
-<span class="sourceLineNo">233</span>  private boolean isBaseZnodeAclSetup(List&lt;ACL&gt; acls) throws IOException {<a name="line.233"></a>
-<span class="sourceLineNo">234</span>    if (LOG.isDebugEnabled()) {<a name="line.234"></a>
-<span class="sourceLineNo">235</span>      LOG.debug("Checking znode ACLs");<a name="line.235"></a>
-<span class="sourceLineNo">236</span>    }<a name="line.236"></a>
-<span class="sourceLineNo">237</span>    String[] superUsers = conf.getStrings(Superusers.SUPERUSER_CONF_KEY);<a name="line.237"></a>
-<span class="sourceLineNo">238</span>    // Check whether ACL set for all superusers<a name="line.238"></a>
-<span class="sourceLineNo">239</span>    if (superUsers != null &amp;&amp; !checkACLForSuperUsers(superUsers, acls)) {<a name="line.239"></a>
-<span class="sourceLineNo">240</span>      return false;<a name="line.240"></a>
-<span class="sourceLineNo">241</span>    }<a name="line.241"></a>
-<span class="sourceLineNo">242</span><a name="line.242"></a>
-<span class="sourceLineNo">243</span>    // this assumes that current authenticated user is the same as zookeeper client user<a name="line.243"></a>
-<span class="sourceLineNo">244</span>    // configured via JAAS<a name="line.244"></a>
-<span class="sourceLineNo">245</span>    String hbaseUser = UserGroupInformation.getCurrentUser().getShortUserName();<a name="line.245"></a>
-<span class="sourceLineNo">246</span><a name="line.246"></a>
-<span class="sourceLineNo">247</span>    if (acls.isEmpty()) {<a name="line.247"></a>
-<span class="sourceLineNo">248</span>      if (LOG.isDebugEnabled()) {<a name="line.248"></a>
-<span class="sourceLineNo">249</span>        LOG.debug("ACL is empty");<a name="line.249"></a>
-<span class="sourceLineNo">250</span>      }<a name="line.250"></a>
-<span class="sourceLineNo">251</span>      return false;<a name="line.251"></a>
-<span class="sourceLineNo">252</span>    }<a name="line.252"></a>
-<span class="sourceLineNo">253</span><a name="line.253"></a>
-<span class="sourceLineNo">254</span>    for (ACL acl : acls) {<a name="line.254"></a>
-<span class="sourceLineNo">255</span>      int perms = acl.getPerms();<a name="line.255"></a>
-<span class="sourceLineNo">256</span>      Id id = acl.getId();<a name="line.256"></a>
-<span class="sourceLineNo">257</span>      // We should only set at most 3 possible ACLs for 3 Ids. One for everyone, one for superuser<a name="line.257"></a>
-<span class="sourceLineNo">258</span>      // and one for the hbase user<a name="line.258"></a>
-<span class="sourceLineNo">259</span>      if (Ids.ANYONE_ID_UNSAFE.equals(id)) {<a name="line.259"></a>
-<span class="sourceLineNo">260</span>        if (perms != Perms.READ) {<a name="line.260"></a>
-<span class="sourceLineNo">261</span>          if (LOG.isDebugEnabled()) {<a name="line.261"></a>
-<span class="sourceLineNo">262</span>            LOG.debug(String.format("permissions for '%s' are not correct: have 0x%x, want 0x%x",<a name="line.262"></a>
-<span class="sourceLineNo">263</span>              id, perms, Perms.READ));<a name="line.263"></a>
-<span class="sourceLineNo">264</span>          }<a name="line.264"></a>
-<span class="sourceLineNo">265</span>          return false;<a name="line.265"></a>
-<span class="sourceLineNo">266</span>        }<a name="line.266"></a>
-<span class="sourceLineNo">267</span>      } else if (superUsers != null &amp;&amp; isSuperUserId(superUsers, id)) {<a name="line.267"></a>
-<span class="sourceLineNo">268</span>        if (perms != Perms.ALL) {<a name="line.268"></a>
-<span class="sourceLineNo">269</span>          if (LOG.isDebugEnabled()) {<a name="line.269"></a>
-<span class="sourceLineNo">270</span>            LOG.debug(String.format("permissions for '%s' are not correct: have 0x%x, want 0x%x",<a name="line.270"></a>
-<span class="sourceLineNo">271</span>              id, perms, Perms.ALL));<a name="line.271"></a>
-<span class="sourceLineNo">272</span>          }<a name="line.272"></a>
-<span class="sourceLineNo">273</span>          return false;<a name="line.273"></a>
-<span class="sourceLineNo">274</span>        }<a name="line.274"></a>
-<span class="sourceLineNo">275</span>      } else if ("sasl".equals(id.getScheme())) {<a name="line.275"></a>
-<span class="sourceLineNo">276</span>        String name = id.getId();<a name="line.276"></a>
-<span class="sourceLineNo">277</span>        // If ZooKeeper recorded the Kerberos full name in the ACL, use only the shortname<a name="line.277"></a>
-<span class="sourceLineNo">278</span>        Matcher match = NAME_PATTERN.matcher(name);<a name="line.278"></a>
-<span class="sourceLineNo">279</span>        if (match.matches()) {<a name="line.279"></a>
-<span class="sourceLineNo">280</span>          name = match.group(1);<a name="line.280"></a>
-<span class="sourceLineNo">281</span>        }<a name="line.281"></a>
-<span class="sourceLineNo">282</span>        if (name.equals(hbaseUser)) {<a name="line.282"></a>
-<span class="sourceLineNo">283</span>          if (perms != Perms.ALL) {<a name="line.283"></a>
-<span class="sourceLineNo">284</span>            if (LOG.isDebugEnabled()) {<a name="line.284"></a>
-<span class="sourceLineNo">285</span>              LOG.debug(String.format("permissions for '%s' are not correct: have 0x%x, want 0x%x",<a name="line.285"></a>
-<span class="sourceLineNo">286</span>                id, perms, Perms.ALL));<a name="line.286"></a>
-<span class="sourceLineNo">287</span>            }<a name="line.287"></a>
-<span class="sourceLineNo">288</span>            return false;<a name="line.288"></a>
-<span class="sourceLineNo">289</span>          }<a name="line.289"></a>
-<span class="sourceLineNo">290</span>        } else {<a name="line.290"></a>
-<span class="sourceLineNo">291</span>          if (LOG.isDebugEnabled()) {<a name="line.291"></a>
-<span class="sourceLineNo">292</span>            LOG.debug("Unexpected shortname in SASL ACL: " + id);<a name="line.292"></a>
-<span class="sourceLineNo">293</span>          }<a name="line.293"></a>
-<span class="sourceLineNo">294</span>          return false;<a name="line.294"></a>
-<span class="sourceLineNo">295</span>        }<a name="line.295"></a>
-<span class="sourceLineNo">296</span>      } else {<a name="line.296"></a>
-<span class="sourceLineNo">297</span>        if (LOG.isDebugEnabled()) {<a name="line.297"></a>
-<span class="sourceLineNo">298</span>          LOG.debug("unexpected ACL id '" + id + "'");<a name="line.298"></a>
-<span class="sourceLineNo">299</span>        }<a name="line.299"></a>
-<span class="sourceLineNo">300</span>        return false;<a name="line.300"></a>
-<span class="sourceLineNo">301</span>      }<a name="line.301"></a>
-<span class="sourceLineNo">302</span>    }<a name="line.302"></a>
-<span class="sourceLineNo">303</span>    return true;<a name="line.303"></a>
-<span class="sourceLineNo">304</span>  }<a name="line.304"></a>
-<span class="sourceLineNo">305</span><a name="line.305"></a>
-<span class="sourceLineNo">306</span>  /*<a name="line.306"></a>
-<span class="sourceLineNo">307</span>   * Validate whether ACL set for all superusers.<a name="line.307"></a>
-<span class="sourceLineNo">308</span>   */<a name="line.308"></a>
-<span class="sourceLineNo">309</span>  private boolean checkACLForSuperUsers(String[] superUsers, List&lt;ACL&gt; acls) {<a name="line.309"></a>
-<span class="sourceLineNo">310</span>    for (String user : superUsers) {<a name="line.310"></a>
-<span class="sourceLineNo">311</span>      boolean hasAccess = false;<a name="line.311"></a>
-<span class="sourceLineNo">312</span>      // TODO: Validate super group members also when ZK supports setting node ACL for groups.<a name="line.312"></a>
-<span class="sourceLineNo">313</span>      if (!AuthUtil.isGroupPrincipal(user)) {<a name="line.313"></a>
-<span class="sourceLineNo">314</span>        for (ACL acl : acls) {<a name="line.314"></a>
-<span class="sourceLineNo">315</span>          if (user.equals(acl.getId().getId())) {<a name="line.315"></a>
-<span class="sourceLineNo">316</span>            if (acl.getPerms() == Perms.ALL) {<a name="line.316"></a>
-<span class="sourceLineNo">317</span>              hasAccess = true;<a name="line.317"></a>
-<span class="sourceLineNo">318</span>            } else {<a name="line.318"></a>
-<span class="sourceLineNo">319</span>              if (LOG.isDebugEnabled()) {<a name="line.319"></a>
-<span class="sourceLineNo">320</span>                LOG.debug(String.format(<a name="line.320"></a>
-<span class="sourceLineNo">321</span>                  "superuser '%s' does not have correct permissions: have 0x%x, want 0x%x",<a name="line.321"></a>
-<span class="sourceLineNo">322</span>                  acl.getId().getId(), acl.getPerms(), Perms.ALL));<a name="line.322"></a>
-<span class="sourceLineNo">323</span>              }<a name="line.323"></a>
-<span class="sourceLineNo">324</span>            }<a name="line.324"></a>
-<span class="sourceLineNo">325</span>            break;<a name="line.325"></a>
-<span class="sourceLineNo">326</span>          }<a name="line.326"></a>
-<span class="sourceLineNo">327</span>        }<a name="line.327"></a>
-<span class="sourceLineNo">328</span>        if (!hasAccess) {<a name="line.328"></a>
-<span class="sourceLineNo">329</span>          return false;<a name="line.329"></a>
-<span class="sourceLineNo">330</span>        }<a name="line.330"></a>
-<span class="sourceLineNo">331</span>      }<a name="line.331"></a>
-<span class="sourceLineNo">332</span>    }<a name="line.332"></a>
-<span class="sourceLineNo">333</span>    return true;<a name="line.333"></a>
-<span class="sourceLineNo">334</span>  }<a name="line.334"></a>
-<span class="sourceLineNo">335</span><a name="line.335"></a>
-<span class="sourceLineNo">336</span>  /*<a name="line.336"></a>
-<span class="sourceLineNo">337</span>   * Validate whether ACL ID is superuser.<a name="line.337"></a>
-<span class="sourceLineNo">338</span>   */<a name="line.338"></a>
-<span class="sourceLineNo">339</span>  public static boolean isSuperUserId(String[] superUsers, Id id) {<a name="line.339"></a>
-<span class="sourceLineNo">340</span>    for (String user : superUsers) {<a name="line.340"></a>
-<span class="sourceLineNo">341</span>      // TODO: Validate super group members also when ZK supports setting node ACL for groups.<a name="line.341"></a>
-<span class="sourceLineNo">342</span>      if (!AuthUtil.isGroupPrincipal(user) &amp;&amp; new Id("sasl", user).equals(id)) {<a name="line.342"></a>
-<span class="sourceLineNo">343</span>        return true;<a name="line.343"></a>
-<span class="sourceLineNo">344</span>      }<a name="line.344"></a>
-<span class="sourceLineNo">345</span>    }<a name="line.345"></a>
-<span class="sourceLineNo">346</span>    return false;<a name="line.346"></a>
-<span class="sourceLineNo">347</span>  }<a name="line.347"></a>
-<span class="sourceLineNo">348</span><a name="line.348"></a>
-<span class="sourceLineNo">349</span>  @Override<a name="line.349"></a>
-<span class="sourceLineNo">350</span>  public String toString() {<a name="line.350"></a>
-<span class="sourceLineNo">351</span>    return this.identifier + ", quorum=" + quorum + ", baseZNode=" + znodePaths.baseZNode;<a name="line.351"></a>
-<span class="sourceLineNo">352</span>  }<a name="line.352"></a>
-<span class="sourceLineNo">353</span><a name="line.353"></a>
-<span class="sourceLineNo">354</span>  /**<a name="line.354"></a>
-<span class="sourceLineNo">355</span>   * Adds this instance's identifier as a prefix to the passed &lt;code&gt;str&lt;/code&gt;<a name="line.355"></a>
-<span class="sourceLineNo">356</span>   * @param str String to amend.<a name="line.356"></a>
-<span class="sourceLineNo">357</span>   * @return A new string with this instance's identifier as prefix: e.g.<a name="line.357"></a>
-<span class="sourceLineNo">358</span>   * if passed 'hello world', the returned string could be<a name="line.358"></a>
-<span class="sourceLineNo">359</span>   */<a name="line.359"></a>
-<span class="sourceLineNo">360</span>  public String prefix(final String str) {<a name="line.360"></a>
-<span class="sourceLineNo">361</span>    return this.toString() + " " + str;<a name="line.361"></a>
-<span class="sourceLineNo">362</span>  }<a name="line.362"></a>
-<span class="sourceLineNo">363</span><a name="line.363"></a>
-<span class="sourceLineNo">364</span>  /**<a name="line.364"></a>
-<span class="sourceLineNo">365</span>   * Get the znodes corresponding to the meta replicas from ZK<a name="line.365"></a>
-<span class="sourceLineNo">366</span>   * @return list of znodes<a name="line.366"></a>
-<span class="sourceLineNo">367</span>   * @throws KeeperException<a name="line.367"></a>
-<span class="sourceLineNo">368</span>   */<a name="line.368"></a>
-<span class="sourceLineNo">369</span>  public List&lt;String&gt; getMetaReplicaNodes() throws KeeperException {<a name="line.369"></a>
-<span class="sourceLineNo">370</span>    List&lt;String&gt; childrenOfBaseNode = ZKUtil.listChildrenNoWatch(this, znodePaths.baseZNode);<a name="line.370"></a>
-<span class="sourceLineNo">371</span>    List&lt;String&gt; metaReplicaNodes = new ArrayList&lt;&gt;(2);<a name="line.371"></a>
-<span class="sourceLineNo">372</span>    if (childrenOfBaseNode != null) {<a name="line.372"></a>
-<span class="sourceLineNo">373</span>      String pattern = conf.get("zookeeper.znode.metaserver","meta-region-server");<a name="line.373"></a>
-<span class="sourceLineNo">374</span>      for (String child : childrenOfBaseNode) {<a name="line.374"></a>
-<span class="sourceLineNo">375</span>        if (child.startsWith(pattern)) metaReplicaNodes.add(child);<a name="line.375"></a>
+<span class="sourceLineNo">090</span>  /* A pattern that matches a Kerberos name, borrowed from Hadoop's KerberosName */<a name="line.90"></a>
+<span class="sourceLineNo">091</span>  private static final Pattern NAME_PATTERN = Pattern.compile("([^/@]*)(/([^/@]*))?@([^/@]*)");<a name="line.91"></a>
+<span class="sourceLineNo">092</span><a name="line.92"></a>
+<span class="sourceLineNo">093</span>  /**<a name="line.93"></a>
+<span class="sourceLineNo">094</span>   * Instantiate a ZooKeeper connection and watcher.<a name="line.94"></a>
+<span class="sourceLineNo">095</span>   * @param identifier string that is passed to RecoverableZookeeper to be used as<a name="line.95"></a>
+<span class="sourceLineNo">096</span>   *                   identifier for this instance. Use null for default.<a name="line.96"></a>
+<span class="sourceLineNo">097</span>   * @throws IOException if the connection to ZooKeeper fails<a name="line.97"></a>
+<span class="sourceLineNo">098</span>   * @throws ZooKeeperConnectionException<a name="line.98"></a>
+<span class="sourceLineNo">099</span>   */<a name="line.99"></a>
+<span class="sourceLineNo">100</span>  public ZKWatcher(Configuration conf, String identifier,<a name="line.100"></a>
+<span class="sourceLineNo">101</span>                   Abortable abortable) throws ZooKeeperConnectionException, IOException {<a name="line.101"></a>
+<span class="sourceLineNo">102</span>    this(conf, identifier, abortable, false);<a name="line.102"></a>
+<span class="sourceLineNo">103</span>  }<a name="line.103"></a>
+<span class="sourceLineNo">104</span><a name="line.104"></a>
+<span class="sourceLineNo">105</span>  /**<a name="line.105"></a>
+<span class="sourceLineNo">106</span>   * Instantiate a ZooKeeper connection and watcher.<a name="line.106"></a>
+<span class="sourceLineNo">107</span>   * @param conf the configuration to use<a name="line.107"></a>
+<span class="sourceLineNo">108</span>   * @param identifier string that is passed to RecoverableZookeeper to be used as identifier for<a name="line.108"></a>
+<span class="sourceLineNo">109</span>   *          this instance. Use null for default.<a name="line.109"></a>
+<span class="sourceLineNo">110</span>   * @param abortable Can be null if there is on error there is no host to abort: e.g. client<a name="line.110"></a>
+<span class="sourceLineNo">111</span>   *          context.<a name="line.111"></a>
+<span class="sourceLineNo">112</span>   * @param canCreateBaseZNode true if a base ZNode can be created<a name="line.112"></a>
+<span class="sourceLineNo">113</span>   * @throws IOException if the connection to ZooKeeper fails<a name="line.113"></a>
+<span class="sourceLineNo">114</span>   * @throws ZooKeeperConnectionException<a name="line.114"></a>
+<span class="sourceLineNo">115</span>   */<a name="line.115"></a>
+<span class="sourceLineNo">116</span>  public ZKWatcher(Configuration conf, String identifier,<a name="line.116"></a>
+<span class="sourceLineNo">117</span>                   Abortable abortable, boolean canCreateBaseZNode)<a name="line.117"></a>
+<span class="sourceLineNo">118</span>  throws IOException, ZooKeeperConnectionException {<a name="line.118"></a>
+<span class="sourceLineNo">119</span>    this.conf = conf;<a name="line.119"></a>
+<span class="sourceLineNo">120</span>    this.quorum = ZKConfig.getZKQuorumServersString(conf);<a name="line.120"></a>
+<span class="sourceLineNo">121</span>    this.prefix = identifier;<a name="line.121"></a>
+<span class="sourceLineNo">122</span>    // Identifier will get the sessionid appended later below down when we<a name="line.122"></a>
+<span class="sourceLineNo">123</span>    // handle the syncconnect event.<a name="line.123"></a>
+<span class="sourceLineNo">124</span>    this.identifier = identifier + "0x0";<a name="line.124"></a>
+<span class="sourceLineNo">125</span>    this.abortable = abortable;<a name="line.125"></a>
+<span class="sourceLineNo">126</span>    this.znodePaths = new ZNodePaths(conf);<a name="line.126"></a>
+<span class="sourceLineNo">127</span>    PendingWatcher pendingWatcher = new PendingWatcher();<a name="line.127"></a>
+<span class="sourceLineNo">128</span>    this.recoverableZooKeeper = ZKUtil.connect(conf, quorum, pendingWatcher, identifier);<a name="line.128"></a>
+<span class="sourceLineNo">129</span>    pendingWatcher.prepare(this);<a name="line.129"></a>
+<span class="sourceLineNo">130</span>    if (canCreateBaseZNode) {<a name="line.130"></a>
+<span class="sourceLineNo">131</span>      try {<a name="line.131"></a>
+<span class="sourceLineNo">132</span>        createBaseZNodes();<a name="line.132"></a>
+<span class="sourceLineNo">133</span>      } catch (ZooKeeperConnectionException zce) {<a name="line.133"></a>
+<span class="sourceLineNo">134</span>        try {<a name="line.134"></a>
+<span class="sourceLineNo">135</span>          this.recoverableZooKeeper.close();<a name="line.135"></a>
+<span class="sourceLineNo">136</span>        } catch (InterruptedException ie) {<a name="line.136"></a>
+<span class="sourceLineNo">137</span>          LOG.debug("Encountered InterruptedException when closing " + this.recoverableZooKeeper);<a name="line.137"></a>
+<span class="sourceLineNo">138</span>          Thread.currentThread().interrupt();<a name="line.138"></a>
+<span class="sourceLineNo">139</span>        }<a name="line.139"></a>
+<span class="sourceLineNo">140</span>        throw zce;<a name="line.140"></a>
+<span class="sourceLineNo">141</span>      }<a name="line.141"></a>
+<span class="sourceLineNo">142</span>    }<a name="line.142"></a>
+<span class="sourceLineNo">143</span>  }<a name="line.143"></a>
+<span class="sourceLineNo">144</span><a name="line.144"></a>
+<span class="sourceLineNo">145</span>  private void createBaseZNodes() throws ZooKeeperConnectionException {<a name="line.145"></a>
+<span class="sourceLineNo">146</span>    try {<a name="line.146"></a>
+<span class="sourceLineNo">147</span>      // Create all the necessary "directories" of znodes<a name="line.147"></a>
+<span class="sourceLineNo">148</span>      ZKUtil.createWithParents(this, znodePaths.baseZNode);<a name="line.148"></a>
+<span class="sourceLineNo">149</span>      ZKUtil.createAndFailSilent(this, znodePaths.rsZNode);<a name="line.149"></a>
+<span class="sourceLineNo">150</span>      ZKUtil.createAndFailSilent(this, znodePaths.drainingZNode);<a name="line.150"></a>
+<span class="sourceLineNo">151</span>      ZKUtil.createAndFailSilent(this, znodePaths.tableZNode);<a name="line.151"></a>
+<span class="sourceLineNo">152</span>      ZKUtil.createAndFailSilent(this, znodePaths.splitLogZNode);<a name="line.152"></a>
+<span class="sourceLineNo">153</span>      ZKUtil.createAndFailSilent(this, znodePaths.backupMasterAddressesZNode);<a name="line.153"></a>
+<span class="sourceLineNo">154</span>      ZKUtil.createAndFailSilent(this, znodePaths.tableLockZNode);<a name="line.154"></a>
+<span class="sourceLineNo">155</span>      ZKUtil.createAndFailSilent(this, znodePaths.masterMaintZNode);<a name="line.155"></a>
+<span class="sourceLineNo">156</span>    } catch (KeeperException e) {<a name="line.156"></a>
+<span class="sourceLineNo">157</span>      throw new ZooKeeperConnectionException(<a name="line.157"></a>
+<span class="sourceLineNo">158</span>          prefix("Unexpected KeeperException creating base node"), e);<a name="line.158"></a>
+<span class="sourceLineNo">159</span>    }<a name="line.159"></a>
+<span class="sourceLineNo">160</span>  }<a name="line.160"></a>
+<span class="sourceLineNo">161</span><a name="line.161"></a>
+<span class="sourceLineNo">162</span>  /** Returns whether the znode is supposed to be readable by the client<a name="line.162"></a>
+<span class="sourceLineNo">163</span>   * and DOES NOT contain sensitive information (world readable).*/<a name="line.163"></a>
+<span class="sourceLineNo">164</span>  public boolean isClientReadable(String node) {<a name="line.164"></a>
+<span class="sourceLineNo">165</span>    // Developer notice: These znodes are world readable. DO NOT add more znodes here UNLESS<a name="line.165"></a>
+<span class="sourceLineNo">166</span>    // all clients need to access this data to work. Using zk for sharing data to clients (other<a name="line.166"></a>
+<span class="sourceLineNo">167</span>    // than service lookup case is not a recommended design pattern.<a name="line.167"></a>
+<span class="sourceLineNo">168</span>    return<a name="line.168"></a>
+<span class="sourceLineNo">169</span>        node.equals(znodePaths.baseZNode) ||<a name="line.169"></a>
+<span class="sourceLineNo">170</span>        znodePaths.isAnyMetaReplicaZNode(node) ||<a name="line.170"></a>
+<span class="sourceLineNo">171</span>        node.equals(znodePaths.masterAddressZNode) ||<a name="line.171"></a>
+<span class="sourceLineNo">172</span>        node.equals(znodePaths.clusterIdZNode)||<a name="line.172"></a>
+<span class="sourceLineNo">173</span>        node.equals(znodePaths.rsZNode) ||<a name="line.173"></a>
+<span class="sourceLineNo">174</span>        // /hbase/table and /hbase/table/foo is allowed, /hbase/table-lock is not<a name="line.174"></a>
+<span class="sourceLineNo">175</span>        node.equals(znodePaths.tableZNode) ||<a name="line.175"></a>
+<span class="sourceLineNo">176</span>        node.startsWith(znodePaths.tableZNode + "/");<a name="line.176"></a>
+<span class="sourceLineNo">177</span>  }<a name="line.177"></a>
+<span class="sourceLineNo">178</span><a name="line.178"></a>
+<span class="sourceLineNo">179</span>  /**<a name="line.179"></a>
+<span class="sourceLineNo">180</span>   * On master start, we check the znode ACLs under the root directory and set the ACLs properly<a name="line.180"></a>
+<span class="sourceLineNo">181</span>   * if needed. If the cluster goes from an unsecure setup to a secure setup, this step is needed<a name="line.181"></a>
+<span class="sourceLineNo">182</span>   * so that the existing znodes created with open permissions are now changed with restrictive<a name="line.182"></a>
+<span class="sourceLineNo">183</span>   * perms.<a name="line.183"></a>
+<span class="sourceLineNo">184</span>   */<a name="line.184"></a>
+<span class="sourceLineNo">185</span>  public void checkAndSetZNodeAcls() {<a name="line.185"></a>
+<span class="sourceLineNo">186</span>    if (!ZKUtil.isSecureZooKeeper(getConfiguration())) {<a name="line.186"></a>
+<span class="sourceLineNo">187</span>      LOG.info("not a secure deployment, proceeding");<a name="line.187"></a>
+<span class="sourceLineNo">188</span>      return;<a name="line.188"></a>
+<span class="sourceLineNo">189</span>    }<a name="line.189"></a>
+<span class="sourceLineNo">190</span><a name="line.190"></a>
+<span class="sourceLineNo">191</span>    // Check the base znodes permission first. Only do the recursion if base znode's perms are not<a name="line.191"></a>
+<span class="sourceLineNo">192</span>    // correct.<a name="line.192"></a>
+<span class="sourceLineNo">193</span>    try {<a name="line.193"></a>
+<span class="sourceLineNo">194</span>      List&lt;ACL&gt; actualAcls = recoverableZooKeeper.getAcl(znodePaths.baseZNode, new Stat());<a name="line.194"></a>
+<span class="sourceLineNo">195</span><a name="line.195"></a>
+<span class="sourceLineNo">196</span>      if (!isBaseZnodeAclSetup(actualAcls)) {<a name="line.196"></a>
+<span class="sourceLineNo">197</span>        LOG.info("setting znode ACLs");<a name="line.197"></a>
+<span class="sourceLineNo">198</span>        setZnodeAclsRecursive(znodePaths.baseZNode);<a name="line.198"></a>
+<span class="sourceLineNo">199</span>      }<a name="line.199"></a>
+<span class="sourceLineNo">200</span>    } catch(KeeperException.NoNodeException nne) {<a name="line.200"></a>
+<span class="sourceLineNo">201</span>      return;<a name="line.201"></a>
+<span class="sourceLineNo">202</span>    } catch(InterruptedException ie) {<a name="line.202"></a>
+<span class="sourceLineNo">203</span>      interruptedExceptionNoThrow(ie, false);<a name="line.203"></a>
+<span class="sourceLineNo">204</span>    } catch (IOException|KeeperException e) {<a name="line.204"></a>
+<span class="sourceLineNo">205</span>      LOG.warn("Received exception while checking and setting zookeeper ACLs", e);<a name="line.205"></a>
+<span class="sourceLineNo">206</span>    }<a name="line.206"></a>
+<span class="sourceLineNo">207</span>  }<a name="line.207"></a>
+<span class="sourceLineNo">208</span><a name="line.208"></a>
+<span class="sourceLineNo">209</span>  /**<a name="line.209"></a>
+<span class="sourceLineNo">210</span>   * Set the znode perms recursively. This will do post-order recursion, so that baseZnode ACLs<a name="line.210"></a>
+<span class="sourceLineNo">211</span>   * will be set last in case the master fails in between.<a name="line.211"></a>
+<span class="sourceLineNo">212</span>   * @param znode the ZNode to set the permissions for<a name="line.212"></a>
+<span class="sourceLineNo">213</span>   */<a name="line.213"></a>
+<span class="sourceLineNo">214</span>  private void setZnodeAclsRecursive(String znode) throws KeeperException, InterruptedException {<a name="line.214"></a>
+<span class="sourceLineNo">215</span>    List&lt;String&gt; children = recoverableZooKeeper.getChildren(znode, false);<a name="line.215"></a>
+<span class="sourceLineNo">216</span><a name="line.216"></a>
+<span class="sourceLineNo">217</span>    for (String child : children) {<a name="line.217"></a>
+<span class="sourceLineNo">218</span>      setZnodeAclsRecursive(ZNodePaths.joinZNode(znode, child));<a name="line.218"></a>
+<span class="sourceLineNo">219</span>    }<a name="line.219"></a>
+<span class="sourceLineNo">220</span>    List&lt;ACL&gt; acls = ZKUtil.createACL(this, znode, true);<a name="line.220"></a>
+<span class="sourceLineNo">221</span>    LOG.info("Setting ACLs for znode:" + znode + " , acl:" + acls);<a name="line.221"></a>
+<span class="sourceLineNo">222</span>    recoverableZooKeeper.setAcl(znode, acls, -1);<a name="line.222"></a>
+<span class="sourceLineNo">223</span>  }<a name="line.223"></a>
+<span class="sourceLineNo">224</span><a name="line.224"></a>
+<span class="sourceLineNo">225</span>  /**<a name="line.225"></a>
+<span class="sourceLineNo">226</span>   * Checks whether the ACLs returned from the base znode (/hbase) is set for secure setup.<a name="line.226"></a>
+<span class="sourceLineNo">227</span>   * @param acls acls from zookeeper<a name="line.227"></a>
+<span class="sourceLineNo">228</span>   * @return whether ACLs are set for the base znode<a name="line.228"></a>
+<span class="sourceLineNo">229</span>   * @throws IOException if getting the current user fails<a name="line.229"></a>
+<span class="sourceLineNo">230</span>   */<a name="line.230"></a>
+<span class="sourceLineNo">231</span>  private boolean isBaseZnodeAclSetup(List&lt;ACL&gt; acls) throws IOException {<a name="line.231"></a>
+<span class="sourceLineNo">232</span>    if (LOG.isDebugEnabled()) {<a name="line.232"></a>
+<span class="sourceLineNo">233</span>      LOG.debug("Checking znode ACLs");<a name="line.233"></a>
+<span class="sourceLineNo">234</span>    }<a name="line.234"></a>
+<span class="sourceLineNo">235</span>    String[] superUsers = conf.getStrings(Superusers.SUPERUSER_CONF_KEY);<a name="line.235"></a>
+<span class="sourceLineNo">236</span>    // Check whether ACL set for all superusers<a name="line.236"></a>
+<span class="sourceLineNo">237</span>    if (superUsers != null &amp;&amp; !checkACLForSuperUsers(superUsers, acls)) {<a name="line.237"></a>
+<span class="sourceLineNo">238</span>      return false;<a name="line.238"></a>
+<span class="sourceLineNo">239</span>    }<a name="line.239"></a>
+<span class="sourceLineNo">240</span><a name="line.240"></a>
+<span class="sourceLineNo">241</span>    // this assumes that current authenticated user is the same as zookeeper client user<a name="line.241"></a>
+<span class="sourceLineNo">242</span>    // configured via JAAS<a name="line.242"></a>
+<span class="sourceLineNo">243</span>    String hbaseUser = UserGroupInformation.getCurrentUser().getShortUserName();<a name="line.243"></a>
+<span class="sourceLineNo">244</span><a name="line.244"></a>
+<span class="sourceLineNo">245</span>    if (acls.isEmpty()) {<a name="line.245"></a>
+<span class="sourceLineNo">246</span>      if (LOG.isDebugEnabled()) {<a name="line.246"></a>
+<span class="sourceLineNo">247</span>        LOG.debug("ACL is empty");<a name="line.247"></a>
+<span class="sourceLineNo">248</span>      }<a name="line.248"></a>
+<span class="sourceLineNo">249</span>      return false;<a name="line.249"></a>
+<span class="sourceLineNo">250</span>    }<a name="line.250"></a>
+<span class="sourceLineNo">251</span><a name="line.251"></a>
+<span class="sourceLineNo">252</span>    for (ACL acl : acls) {<a name="line.252"></a>
+<span class="sourceLineNo">253</span>      int perms = acl.getPerms();<a name="line.253"></a>
+<span class="sourceLineNo">254</span>      Id id = acl.getId();<a name="line.254"></a>
+<span class="sourceLineNo">255</span>      // We should only set at most 3 possible ACLs for 3 Ids. One for everyone, one for superuser<a name="line.255"></a>
+<span class="sourceLineNo">256</span>      // and one for the hbase user<a name="line.256"></a>
+<span class="sourceLineNo">257</span>      if (Ids.ANYONE_ID_UNSAFE.equals(id)) {<a name="line.257"></a>
+<span class="sourceLineNo">258</span>        if (perms != Perms.READ) {<a name="line.258"></a>
+<span class="sourceLineNo">259</span>          if (LOG.isDebugEnabled()) {<a name="line.259"></a>
+<span class="sourceLineNo">260</span>            LOG.debug(String.format("permissions for '%s' are not correct: have 0x%x, want 0x%x",<a name="line.260"></a>
+<span class="sourceLineNo">261</span>              id, perms, Perms.READ));<a name="line.261"></a>
+<span class="sourceLineNo">262</span>          }<a name="line.262"></a>
+<span class="sourceLineNo">263</span>          return false;<a name="line.263"></a>
+<span class="sourceLineNo">264</span>        }<a name="line.264"></a>
+<span class="sourceLineNo">265</span>      } else if (superUsers != null &amp;&amp; isSuperUserId(superUsers, id)) {<a name="line.265"></a>
+<span class="sourceLineNo">266</span>        if (perms != Perms.ALL) {<a name="line.266"></a>
+<span class="sourceLineNo">267</span>          if (LOG.isDebugEnabled()) {<a name="line.267"></a>
+<span class="sourceLineNo">268</span>            LOG.debug(String.format("permissions for '%s' are not correct: have 0x%x, want 0x%x",<a name="line.268"></a>
+<span class="sourceLineNo">269</span>              id, perms, Perms.ALL));<a name="line.269"></a>
+<span class="sourceLineNo">270</span>          }<a name="line.270"></a>
+<span class="sourceLineNo">271</span>          return false;<a name="line.271"></a>
+<span class="sourceLineNo">272</span>        }<a name="line.272"></a>
+<span class="sourceLineNo">273</span>      } else if ("sasl".equals(id.getScheme())) {<a name="line.273"></a>
+<span class="sourceLineNo">274</span>        String name = id.getId();<a name="line.274"></a>
+<span class="sourceLineNo">275</span>        // If ZooKeeper recorded the Kerberos full name in the ACL, use only the shortname<a name="line.275"></a>
+<span class="sourceLineNo">276</span>        Matcher match = NAME_PATTERN.matcher(name);<a name="line.276"></a>
+<span class="sourceLineNo">277</span>        if (match.matches()) {<a name="line.277"></a>
+<span class="sourceLineNo">278</span>          name = match.group(1);<a name="line.278"></a>
+<span class="sourceLineNo">279</span>        }<a name="line.279"></a>
+<span class="sourceLineNo">280</span>        if (name.equals(hbaseUser)) {<a name="line.280"></a>
+<span class="sourceLineNo">281</span>          if (perms != Perms.ALL) {<a name="line.281"></a>
+<span class="sourceLineNo">282</span>            if (LOG.isDebugEnabled()) {<a name="line.282"></a>
+<span class="sourceLineNo">283</span>              LOG.debug(String.format("permissions for '%s' are not correct: have 0x%x, want 0x%x",<a name="line.283"></a>
+<span class="sourceLineNo">284</span>                id, perms, Perms.ALL));<a name="line.284"></a>
+<span class="sourceLineNo">285</span>            }<a name="line.285"></a>
+<span class="sourceLineNo">286</span>            return false;<a name="line.286"></a>
+<span class="sourceLineNo">287</span>          }<a name="line.287"></a>
+<span class="sourceLineNo">288</span>        } else {<a name="line.288"></a>
+<span class="sourceLineNo">289</span>          if (LOG.isDebugEnabled()) {<a name="line.289"></a>
+<span class="sourceLineNo">290</span>            LOG.debug("Unexpected shortname in SASL ACL: " + id);<a name="line.290"></a>
+<span class="sourceLineNo">291</span>          }<a name="line.291"></a>
+<span class="sourceLineNo">292</span>          return false;<a name="line.292"></a>
+<span class="sourceLineNo">293</span>        }<a name="line.293"></a>
+<span class="sourceLineNo">294</span>      } else {<a name="line.294"></a>
+<span class="sourceLineNo">295</span>        if (LOG.isDebugEnabled()) {<a name="line.295"></a>
+<span class="sourceLineNo">296</span>          LOG.debug("unexpected ACL id '" + id + "'");<a name="line.296"></a>
+<span class="sourceLineNo">297</span>        }<a name="line.297"></a>
+<span class="sourceLineNo">298</span>        return false;<a name="line.298"></a>
+<span class="sourceLineNo">299</span>      }<a name="line.299"></a>
+<span class="sourceLineNo">300</span>    }<a name="line.300"></a>
+<span class="sourceLineNo">301</span>    return true;<a name="line.301"></a>
+<span class="sourceLineNo">302</span>  }<a name="line.302"></a>
+<span class="sourceLineNo">303</span><a name="line.303"></a>
+<span class="sourceLineNo">304</span>  /*<a name="line.304"></a>
+<span class="sourceLineNo">305</span>   * Validate whether ACL set for all superusers.<a name="line.305"></a>
+<span class="sourceLineNo">306</span>   */<a name="line.306"></a>
+<span class="sourceLineNo">307</span>  private boolean checkACLForSuperUsers(String[] superUsers, List&lt;ACL&gt; acls) {<a name="line.307"></a>
+<span class="sourceLineNo">308</span>    for (String user : superUsers) {<a name="line.308"></a>
+<span class="sourceLineNo">309</span>      boolean hasAccess = false;<a name="line.309"></a>
+<span class="sourceLineNo">310</span>      // TODO: Validate super group members also when ZK supports setting node ACL for groups.<a name="line.310"></a>
+<span class="sourceLineNo">311</span>      if (!AuthUtil.isGroupPrincipal(user)) {<a name="line.311"></a>
+<span class="sourceLineNo">312</span>        for (ACL acl : acls) {<a name="line.312"></a>
+<span class="sourceLineNo">313</span>          if (user.equals(acl.getId().getId())) {<a name="line.313"></a>
+<span class="sourceLineNo">314</span>            if (acl.getPerms() == Perms.ALL) {<a name="line.314"></a>
+<span class="sourceLineNo">315</span>              hasAccess = true;<a name="line.315"></a>
+<span class="sourceLineNo">316</span>            } else {<a name="line.316"></a>
+<span class="sourceLineNo">317</span>              if (LOG.isDebugEnabled()) {<a name="line.317"></a>
+<span class="sourceLineNo">318</span>                LOG.debug(String.format(<a name="line.318"></a>
+<span class="sourceLineNo">319</span>                  "superuser '%s' does not have correct permissions: have 0x%x, want 0x%x",<a name="line.319"></a>
+<span class="sourceLineNo">320</span>                  acl.getId().getId(), acl.getPerms(), Perms.ALL));<a name="line.320"></a>
+<span class="sourceLineNo">321</span>              }<a name="line.321"></a>
+<span class="sourceLineNo">322</span>            }<a name="line.322"></a>
+<span class="sourceLineNo">323</span>            break;<a name="line.323"></a>
+<span class="sourceLineNo">324</span>          }<a name="line.324"></a>
+<span class="sourceLineNo">325</span>        }<a name="line.325"></a>
+<span class="sourceLineNo">326</span>        if (!hasAccess) {<a name="line.326"></a>
+<span class="sourceLineNo">327</span>          return false;<a name="line.327"></a>
+<span class="sourceLineNo">328</span>        }<a name="line.328"></a>
+<span class="sourceLineNo">329</span>      }<a name="line.329"></a>
+<span class="sourceLineNo">330</span>    }<a name="line.330"></a>
+<span class="sourceLineNo">331</span>    return true;<a name="line.331"></a>
+<span class="sourceLineNo">332</span>  }<a name="line.332"></a>
+<span class="sourceLineNo">333</span><a name="line.333"></a>
+<span class="sourceLineNo">334</span>  /*<a name="line.334"></a>
+<span class="sourceLineNo">335</span>   * Validate whether ACL ID is superuser.<a name="line.335"></a>
+<span class="sourceLineNo">336</span>   */<a name="line.336"></a>
+<span class="sourceLineNo">337</span>  public static boolean isSuperUserId(String[] superUsers, Id id) {<a name="line.337"></a>
+<span class="sourceLineNo">338</span>    for (String user : superUsers) {<a name="line.338"></a>
+<span class="sourceLineNo">339</span>      // TODO: Validate super group members also when ZK supports setting node ACL for groups.<a name="line.339"></a>
+<span class="sourceLineNo">340</span>      if (!AuthUtil.isGroupPrincipal(user) &amp;&amp; new Id("sasl", user).equals(id)) {<a name="line.340"></a>
+<span class="sourceLineNo">341</span>        return true;<a name="line.341"></a>
+<span class="sourceLineNo">342</span>      }<a name="line.342"></a>
+<span class="sourceLineNo">343</span>    }<a name="line.343"></a>
+<span class="sourceLineNo">344</span>    return false;<a name="line.344"></a>
+<span class="sourceLineNo">345</span>  }<a name="line.345"></a>
+<span class="sourceLineNo">346</span><a name="line.346"></a>
+<span class="sourceLineNo">347</span>  @Override<a name="line.347"></a>
+<span class="sourceLineNo">348</span>  public String toString() {<a name="line.348"></a>
+<span class="sourceLineNo">349</span>    return this.identifier + ", quorum=" + quorum + ", baseZNode=" + znodePaths.baseZNode;<a name="line.349"></a>
+<span class="sourceLineNo">350</span>  }<a name="line.350"></a>
+<span class="sourceLineNo">351</span><a name="line.351"></a>
+<span class="sourceLineNo">352</span>  /**<a name="line.352"></a>
+<span class="sourceLineNo">353</span>   * Adds this instance's identifier as a prefix to the passed &lt;code&gt;str&lt;/code&gt;<a name="line.353"></a>
+<span class="sourceLineNo">354</span>   * @param str String to amend.<a name="line.354"></a>
+<span class="sourceLineNo">355</span>   * @return A new string with this instance's identifier as prefix: e.g.<a name="line.355"></a>
+<span class="sourceLineNo">356</span>   *         if passed 'hello world', the returned string could be<a name="line.356"></a>
+<span class="sourceLineNo">357</span>   */<a name="line.357"></a>
+<span class="sourceLineNo">358</span>  public String prefix(final String str) {<a name="line.358"></a>
+<span class="sourceLineNo">359</span>    return this.toString() + " " + str;<a name="line.359"></a>
+<span class="sourceLineNo">360</span>  }<a name="line.360"></a>
+<span class="sourceLineNo">361</span><a name="line.361"></a>
+<span class="sourceLineNo">362</span>  /**<a name="line.362"></a>
+<span class="sourceLineNo">363</span>   * Get the znodes corresponding to the meta replicas from ZK<a name="line.363"></a>
+<span class="sourceLineNo">364</span>   * @return list of znodes<a name="line.364"></a>
+<span class="sourceLineNo">365</span>   * @throws KeeperException if a ZooKeeper operation fails<a name="line.365"></a>
+<span class="sourceLineNo">366</span>   */<a name="line.366"></a>
+<span class="sourceLineNo">367</span>  public List&lt;String&gt; getMetaReplicaNodes() throws KeeperException {<a name="line.367"></a>
+<span class="sourceLineNo">368</span>    List&lt;String&gt; childrenOfBaseNode = ZKUtil.listChildrenNoWatch(this, znodePaths.baseZNode);<a name="line.368"></a>
+<span class="sourceLineNo">369</span>    List&lt;String&gt; metaReplicaNodes = new ArrayList&lt;&gt;(2);<a name="line.369"></a>
+<span class="sourceLineNo">370</span>    if (childrenOfBaseNode != null) {<a name="line.370"></a>
+<span class="sourceLineNo">371</span>      String pattern = conf.get("zookeeper.znode.metaserver","meta-region-server");<a name="line.371"></a>
+<span class="sourceLineNo">372</span>      for (String child : childrenOfBaseNode) {<a name="line.372"></a>
+<span class="sourceLineNo">373</span>        if (child.startsWith(pattern)) {<a name="line.373"></a>
+<span class="sourceLineNo">374</span>          metaReplicaNodes.add(child);<a name="line.374"></a>
+<span class="sourceLineNo">375</span>        }<a name="line.375"></a>
 <span class="sourceLineNo">376</span>      }<a name="line.376"></a>
 <span class="sourceLineNo">377</span>    }<a name="line.377"></a>
 <span class="sourceLineNo">378</span>    return metaReplicaNodes;<a name="line.378"></a>
@@ -388,7 +388,7 @@
 <span class="sourceLineNo">380</span><a name="line.380"></a>
 <span class="sourceLineNo">381</span>  /**<a name="line.381"></a>
 <span class="sourceLineNo">382</span>   * Register the specified listener to receive ZooKeeper events.<a name="line.382"></a>
-<span class="sourceLineNo">383</span>   * @param listener<a name="line.383"></a>
+<span class="sourceLineNo">383</span>   * @param listener the listener to register<a name="line.383"></a>
 <span class="sourceLineNo">384</span>   */<a name="line.384"></a>
 <span class="sourceLineNo">385</span>  public void registerListener(ZKListener listener) {<a name="line.385"></a>
 <span class="sourceLineNo">386</span>    listeners.add(listener);<a name="line.386"></a>
@@ -397,7 +397,7 @@
 <span class="sourceLineNo">389</span>  /**<a name="line.389"></a>
 <span class="sourceLineNo">390</span>   * Register the specified listener to receive ZooKeeper events and add it as<a name="line.390"></a>
 <span class="sourceLineNo">391</span>   * the first in the list of current listeners.<a name="line.391"></a>
-<span class="sourceLineNo">392</span>   * @param listener<a name="line.392"></a>
+<span class="sourceLineNo">392</span>   * @param listener the listener to register<a name="line.392"></a>
 <span class="sourceLineNo">393</span>   */<a name="line.393"></a>
 <span class="sourceLineNo">394</span>  public void registerListenerFirst(ZKListener listener) {<a name="line.394"></a>
 <span class="sourceLineNo">395</span>    listeners.add(0, listener);<a name="line.395"></a>
@@ -520,7 +520,7 @@
 <span class="sourceLineNo">512</span>   * call, it's possible for the Abortable to catch it and try to create a new<a name="line.512"></a>
 <span class="sourceLineNo">513</span>   * session with ZooKeeper. This is what the client does in HCM.<a name="line.513"></a>
 <span class="sourceLineNo">514</span>   * &lt;p&gt;<a name="line.514"></a>
-<span class="sourceLineNo">515</span>   * @param event<a name="line.515"></a>
+<span class="sourceLineNo">515</span>   * @param event the connection-related event<a name="line.515"></a>
 <span class="sourceLineNo">516</span>   */<a name="line.516"></a>
 <span class="sourceLineNo">517</span>  private void connectionEvent(WatchedEvent event) {<a name="line.517"></a>
 <span class="sourceLineNo">518</span>    switch(event.getState()) {<a name="line.518"></a>
@@ -579,67 +579,69 @@
 <span class="sourceLineNo">571</span>   * &lt;p&gt;<a name="line.571"></a>
 <span class="sourceLineNo">572</span>   * TODO: Currently this method rethrows the exception to let the caller handle<a name="line.572"></a>
 <span class="sourceLineNo">573</span>   * &lt;p&gt;<a name="line.573"></a>
-<span class="sourceLineNo">574</span>   * @param ke<a name="line.574"></a>
-<span class="sourceLineNo">575</span>   * @throws KeeperException<a name="line.575"></a>
+<span class="sourceLineNo">574</span>   * @param ke the exception to rethrow<a name="line.574"></a>
+<span class="sourceLineNo">575</span>   * @throws KeeperException if a ZooKeeper operation fails<a name="line.575"></a>
 <span class="sourceLineNo">576</span>   */<a name="line.576"></a>
-<span class="sourceLineNo">577</span>  public void keeperException(KeeperException ke)<a name="line.577"></a>
-<span class="sourceLineNo">578</span>  throws KeeperException {<a name="line.578"></a>
-<span class="sourceLineNo">579</span>    LOG.error(prefix("Received unexpected KeeperException, re-throwing exception"), ke);<a name="line.579"></a>
-<span class="sourceLineNo">580</span>    throw ke;<a name="line.580"></a>
-<span class="sourceLineNo">581</span>  }<a name="line.581"></a>
-<span class="sourceLineNo">582</span><a name="line.582"></a>
-<span class="sourceLineNo">583</span>  /**<a name="line.583"></a>
-<span class="sourceLineNo">584</span>   * Handles InterruptedExceptions in client calls.<a name="line.584"></a>
-<span class="sourceLineNo">585</span>   * @param ie the InterruptedException instance thrown<a name="line.585"></a>
-<span class="sourceLineNo">586</span>   * @throws KeeperException the exception to throw, transformed from the InterruptedException<a name="line.586"></a>
-<span class="sourceLineNo">587</span>   */<a name="line.587"></a>
-<span class="sourceLineNo">588</span>  public void interruptedException(InterruptedException ie) throws KeeperException {<a name="line.588"></a>
-<span class="sourceLineNo">589</span>    interruptedExceptionNoThrow(ie, true);<a name="line.589"></a>
-<span class="sourceLineNo">590</span>    // Throw a system error exception to let upper level handle it<a name="line.590"></a>
-<span class="sourceLineNo">591</span>    throw new KeeperException.SystemErrorException();<a name="line.591"></a>
-<span class="sourceLineNo">592</span>  }<a name="line.592"></a>
-<span class="sourceLineNo">593</span><a name="line.593"></a>
-<span class="sourceLineNo">594</span>  /**<a name="line.594"></a>
-<span class="sourceLineNo">595</span>   * Log the InterruptedException and interrupt current thread<a name="line.595"></a>
-<span class="sourceLineNo">596</span>   * @param ie The IterruptedException to log<a name="line.596"></a>
-<span class="sourceLineNo">597</span>   * @param throwLater Whether we will throw the exception latter<a name="line.597"></a>
-<span class="sourceLineNo">598</span>   */<a name="line.598"></a>
-<span class="sourceLineNo">599</span>  public void interruptedExceptionNoThrow(InterruptedException ie, boolean throwLater) {<a name="line.599"></a>
-<span class="sourceLineNo">600</span>    LOG.debug(prefix("Received InterruptedException, will interrupt current thread"<a name="line.600"></a>
-<span class="sourceLineNo">601</span>        + (throwLater ? " and rethrow a SystemErrorException" : "")),<a name="line.601"></a>
-<span class="sourceLineNo">602</span>      ie);<a name="line.602"></a>
-<span class="sourceLineNo">603</span>    // At least preserve interrupt.<a name="line.603"></a>
-<span class="sourceLineNo">604</span>    Thread.currentThread().interrupt();<a name="line.604"></a>
-<span class="sourceLineNo">605</span>  }<a name="line.605"></a>
-<span class="sourceLineNo">606</span><a name="line.606"></a>
-<span class="sourceLineNo">607</span>  /**<a name="line.607"></a>
-<span class="sourceLineNo">608</span>   * Close the connection to ZooKeeper.<a name="line.608"></a>
-<span class="sourceLineNo">609</span>   *<a name="line.609"></a>
-<span class="sourceLineNo">610</span>   */<a name="line.610"></a>
-<span class="sourceLineNo">611</span>  @Override<a name="line.611"></a>
-<span class="sourceLineNo">612</span>  public void close() {<a name="line.612"></a>
-<span class="sourceLineNo">613</span>    try {<a name="line.613"></a>
-<span class="sourceLineNo">614</span>      recoverableZooKeeper.close();<a name="line.614"></a>
-<span class="sourceLineNo">615</span>    } catch (InterruptedException e) {<a name="line.615"></a>
-<span class="sourceLineNo">616</span>      Thread.currentThread().interrupt();<a name="line.616"></a>
-<span class="sourceLineNo">617</span>    }<a name="line.617"></a>
-<span class="sourceLineNo">618</span>  }<a name="line.618"></a>
-<span class="sourceLineNo">619</span><a name="line.619"></a>
-<span class="sourceLineNo">620</span>  public Configuration getConfiguration() {<a name="line.620"></a>
-<span class="sourceLineNo">621</span>    return conf;<a name="line.621"></a>
-<span class="sourceLineNo">622</span>  }<a name="line.622"></a>
-<span class="sourceLineNo">623</span><a name="line.623"></a>
-<span class="sourceLineNo">624</span>  @Override<a name="line.624"></a>
-<span class="sourceLineNo">625</span>  public void abort(String why, Throwable e) {<a name="line.625"></a>
-<span class="sourceLineNo">626</span>    if (this.abortable != null) this.abortable.abort(why, e);<a name="line.626"></a>
-<span class="sourceLineNo">627</span>    else this.aborted = true;<a name="line.627"></a>
-<span class="sourceLineNo">628</span>  }<a name="line.628"></a>
-<span class="sourceLineNo">629</span><a name="line.629"></a>
-<span class="sourceLineNo">630</span>  @Override<a name="line.630"></a>
-<span class="sourceLineNo">631</span>  public boolean isAborted() {<a name="line.631"></a>
-<span class="sourceLineNo">632</span>    return this.abortable == null? this.aborted: this.abortable.isAborted();<a name="line.632"></a>
-<span class="sourceLineNo">633</span>  }<a name="line.633"></a>
-<span class="sourceLineNo">634</span>}<a name="line.634"></a>
+<span class="sourceLineNo">577</span>  public void keeperException(KeeperException ke) throws KeeperException {<a name="line.577"></a>
+<span class="sourceLineNo">578</span>    LOG.error(prefix("Received unexpected KeeperException, re-throwing exception"), ke);<a name="line.578"></a>
+<span class="sourceLineNo">579</span>    throw ke;<a name="line.579"></a>
+<span class="sourceLineNo">580</span>  }<a name="line.580"></a>
+<span class="sourceLineNo">581</span><a name="line.581"></a>
+<span class="sourceLineNo">582</span>  /**<a name="line.582"></a>
+<span class="sourceLineNo">583</span>   * Handles InterruptedExceptions in client calls.<a name="line.583"></a>
+<span class="sourceLineNo">584</span>   * @param ie the InterruptedException instance thrown<a name="line.584"></a>
+<span class="sourceLineNo">585</span>   * @throws KeeperException the exception to throw, transformed from the InterruptedException<a name="line.585"></a>
+<span class="sourceLineNo">586</span>   */<a name="line.586"></a>
+<span class="sourceLineNo">587</span>  public void interruptedException(InterruptedException ie) throws KeeperException {<a name="line.587"></a>
+<span class="sourceLineNo">588</span>    interruptedExceptionNoThrow(ie, true);<a name="line.588"></a>
+<span class="sourceLineNo">589</span>    // Throw a system error exception to let upper level handle it<a name="line.589"></a>
+<span class="sourceLineNo">590</span>    throw new KeeperException.SystemErrorException();<a name="line.590"></a>
+<span class="sourceLineNo">591</span>  }<a name="line.591"></a>
+<span class="sourceLineNo">592</span><a name="line.592"></a>
+<span class="sourceLineNo">593</span>  /**<a name="line.593"></a>
+<span class="sourceLineNo">594</span>   * Log the InterruptedException and interrupt current thread<a name="line.594"></a>
+<span class="sourceLineNo">595</span>   * @param ie The IterruptedException to log<a name="line.595"></a>
+<span class="sourceLineNo">596</span>   * @param throwLater Whether we will throw the exception latter<a name="line.596"></a>
+<span class="sourceLineNo">597</span>   */<a name="line.597"></a>
+<span class="sourceLineNo">598</span>  public void interruptedExceptionNoThrow(InterruptedException ie, boolean throwLater) {<a name="line.598"></a>
+<span class="sourceLineNo">599</span>    LOG.debug(prefix("Received InterruptedException, will interrupt current thread"<a name="line.599"></a>
+<span class="sourceLineNo">600</span>        + (throwLater ? " and rethrow a SystemErrorException" : "")),<a name="line.600"></a>
+<span class="sourceLineNo">601</span>      ie);<a name="line.601"></a>
+<span class="sourceLineNo">602</span>    // At least preserve interrupt.<a name="line.602"></a>
+<span class="sourceLineNo">603</span>    Thread.currentThread().interrupt();<a name="line.603"></a>
+<span class="sourceLineNo">604</span>  }<a name="line.604"></a>
+<span class="sourceLineNo">605</span><a name="line.605"></a>
+<span class="sourceLineNo">606</span>  /**<a name="line.606"></a>
+<span class="sourceLineNo">607</span>   * Close the connection to ZooKeeper.<a name="line.607"></a>
+<span class="sourceLineNo">608</span>   *<a name="line.608"></a>
+<span class="sourceLineNo">609</span>   */<a name="line.609"></a>
+<span class="sourceLineNo">610</span>  @Override<a name="line.610"></a>
+<span class="sourceLineNo">611</span>  public void close() {<a name="line.611"></a>
+<span class="sourceLineNo">612</span>    try {<a name="line.612"></a>
+<span class="sourceLineNo">613</span>      recoverableZooKeeper.close();<a name="line.613"></a>
+<span class="sourceLineNo">614</span>    } catch (InterruptedException e) {<a name="line.614"></a>
+<span class="sourceLineNo">615</span>      Thread.currentThread().interrupt();<a name="line.615"></a>
+<span class="sourceLineNo">616</span>    }<a name="line.616"></a>
+<span class="sourceLineNo">617</span>  }<a name="line.617"></a>
+<span class="sourceLineNo">618</span><a name="line.618"></a>
+<span class="sourceLineNo">619</span>  public Configuration getConfiguration() {<a name="line.619"></a>
+<span class="sourceLineNo">620</span>    return conf;<a name="line.620"></a>
+<span class="sourceLineNo">621</span>  }<a name="line.621"></a>
+<span class="sourceLineNo">622</span><a name="line.622"></a>
+<span class="sourceLineNo">623</span>  @Override<a name="line.623"></a>
+<span class="sourceLineNo">624</span>  public void abort(String why, Throwable e) {<a name="line.624"></a>
+<span class="sourceLineNo">625</span>    if (this.abortable != null) {<a name="line.625"></a>
+<span class="sourceLineNo">626</span>      this.abortable.abort(why, e);<a name="line.626"></a>
+<span class="sourceLineNo">627</span>    } else {<a name="line.627"></a>
+<span class="sourceLineNo">628</span>      this.aborted = true;<a name="line.628"></a>
+<span class="sourceLineNo">629</span>    }<a name="line.629"></a>
+<span class="sourceLineNo">630</span>  }<a name="line.630"></a>
+<span class="sourceLineNo">631</span><a name="line.631"></a>
+<span class="sourceLineNo">632</span>  @Override<a name="line.632"></a>
+<span class="sourceLineNo">633</span>  public boolean isAborted() {<a name="line.633"></a>
+<span class="sourceLineNo">634</span>    return this.abortable == null? this.aborted: this.abortable.isAborted();<a name="line.634"></a>
+<span class="sourceLineNo">635</span>  }<a name="line.635"></a>
+<span class="sourceLineNo">636</span>}<a name="line.636"></a>
 
 
 

http://git-wip-us.apache.org/repos/asf/hbase-site/blob/48bd3915/export_control.html
----------------------------------------------------------------------
diff --git a/export_control.html b/export_control.html
index 255ce75..09c9fab 100644
--- a/export_control.html
+++ b/export_control.html
@@ -7,7 +7,7 @@
   <head>
     <meta charset="UTF-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <meta name="Date-Revision-yyyymmdd" content="20171216" />
+    <meta name="Date-Revision-yyyymmdd" content="20171217" />
     <meta http-equiv="Content-Language" content="en" />
     <title>Apache HBase &#x2013; 
       Export Control
@@ -336,7 +336,7 @@ for more details.</p>
                         <a href="https://www.apache.org/">The Apache Software Foundation</a>.
             All rights reserved.      
                     
-                  <li id="publishDate" class="pull-right">Last Published: 2017-12-16</li>
+                  <li id="publishDate" class="pull-right">Last Published: 2017-12-17</li>
             </p>
                 </div>
 

http://git-wip-us.apache.org/repos/asf/hbase-site/blob/48bd3915/hbase-annotations/checkstyle.html
----------------------------------------------------------------------
diff --git a/hbase-annotations/checkstyle.html b/hbase-annotations/checkstyle.html
index 8ad19cd..786327e 100644
--- a/hbase-annotations/checkstyle.html
+++ b/hbase-annotations/checkstyle.html
@@ -7,7 +7,7 @@
   <head>
     <meta charset="UTF-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <meta name="Date-Revision-yyyymmdd" content="20171216" />
+    <meta name="Date-Revision-yyyymmdd" content="20171217" />
     <meta http-equiv="Content-Language" content="en" />
     <title>Apache HBase - Annotations &#x2013; Checkstyle Results</title>
     <link rel="stylesheet" href="./css/apache-maven-fluido-1.5-HBASE.min.css" />
@@ -150,7 +150,7 @@
                         <a href="https://www.apache.org/">The Apache Software Foundation</a>.
             All rights reserved.      
                     
-                  <li id="publishDate" class="pull-right">Last Published: 2017-12-16</li>
+                  <li id="publishDate" class="pull-right">Last Published: 2017-12-17</li>
             </p>
                 </div>
 

http://git-wip-us.apache.org/repos/asf/hbase-site/blob/48bd3915/hbase-annotations/dependencies.html
----------------------------------------------------------------------
diff --git a/hbase-annotations/dependencies.html b/hbase-annotations/dependencies.html
index 28e46fb..6c31d9f 100644
--- a/hbase-annotations/dependencies.html
+++ b/hbase-annotations/dependencies.html
@@ -7,7 +7,7 @@
   <head>
     <meta charset="UTF-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <meta name="Date-Revision-yyyymmdd" content="20171216" />
+    <meta name="Date-Revision-yyyymmdd" content="20171217" />
     <meta http-equiv="Content-Language" content="en" />
     <title>Apache HBase - Annotations &#x2013; Project Dependencies</title>
     <link rel="stylesheet" href="./css/apache-maven-fluido-1.5-HBASE.min.css" />
@@ -272,7 +272,7 @@
                         <a href="https://www.apache.org/">The Apache Software Foundation</a>.
             All rights reserved.      
                     
-                  <li id="publishDate" class="pull-right">Last Published: 2017-12-16</li>
+                  <li id="publishDate" class="pull-right">Last Published: 2017-12-17</li>
             </p>
                 </div>
 

http://git-wip-us.apache.org/repos/asf/hbase-site/blob/48bd3915/hbase-annotations/dependency-convergence.html
----------------------------------------------------------------------
diff --git a/hbase-annotations/dependency-convergence.html b/hbase-annotations/dependency-convergence.html
index a5e9095..fc28026 100644
--- a/hbase-annotations/dependency-convergence.html
+++ b/hbase-annotations/dependency-convergence.html
@@ -7,7 +7,7 @@
   <head>
     <meta charset="UTF-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <meta name="Date-Revision-yyyymmdd" content="20171216" />
+    <meta name="Date-Revision-yyyymmdd" content="20171217" />
     <meta http-equiv="Content-Language" content="en" />
     <title>Apache HBase - Annotations &#x2013; Reactor Dependency Convergence</title>
     <link rel="stylesheet" href="./css/apache-maven-fluido-1.5-HBASE.min.css" />
@@ -829,7 +829,7 @@
                         <a href="https://www.apache.org/">The Apache Software Foundation</a>.
             All rights reserved.      
                     
-                  <li id="publishDate" class="pull-right">Last Published: 2017-12-16</li>
+                  <li id="publishDate" class="pull-right">Last Published: 2017-12-17</li>
             </p>
                 </div>
 

http://git-wip-us.apache.org/repos/asf/hbase-site/blob/48bd3915/hbase-annotations/dependency-info.html
----------------------------------------------------------------------
diff --git a/hbase-annotations/dependency-info.html b/hbase-annotations/dependency-info.html
index 13baf3e..36e4d1a 100644
--- a/hbase-annotations/dependency-info.html
+++ b/hbase-annotations/dependency-info.html
@@ -7,7 +7,7 @@
   <head>
     <meta charset="UTF-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <meta name="Date-Revision-yyyymmdd" content="20171216" />
+    <meta name="Date-Revision-yyyymmdd" content="20171217" />
     <meta http-equiv="Content-Language" content="en" />
     <title>Apache HBase - Annotations &#x2013; Dependency Information</title>
     <link rel="stylesheet" href="./css/apache-maven-fluido-1.5-HBASE.min.css" />
@@ -147,7 +147,7 @@
                         <a href="https://www.apache.org/">The Apache Software Foundation</a>.
             All rights reserved.      
                     
-                  <li id="publishDate" class="pull-right">Last Published: 2017-12-16</li>
+                  <li id="publishDate" class="pull-right">Last Published: 2017-12-17</li>
             </p>
                 </div>
 

http://git-wip-us.apache.org/repos/asf/hbase-site/blob/48bd3915/hbase-annotations/dependency-management.html
----------------------------------------------------------------------
diff --git a/hbase-annotations/dependency-management.html b/hbase-annotations/dependency-management.html
index 8cba5c7..5717679 100644
--- a/hbase-annotations/dependency-management.html
+++ b/hbase-annotations/dependency-management.html
@@ -7,7 +7,7 @@
   <head>
     <meta charset="UTF-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <meta name="Date-Revision-yyyymmdd" content="20171216" />
+    <meta name="Date-Revision-yyyymmdd" content="20171217" />
     <meta http-equiv="Content-Language" content="en" />
     <title>Apache HBase - Annotations &#x2013; Project Dependency Management</title>
     <link rel="stylesheet" href="./css/apache-maven-fluido-1.5-HBASE.min.css" />
@@ -810,7 +810,7 @@
                         <a href="https://www.apache.org/">The Apache Software Foundation</a>.
             All rights reserved.      
                     
-                  <li id="publishDate" class="pull-right">Last Published: 2017-12-16</li>
+                  <li id="publishDate" class="pull-right">Last Published: 2017-12-17</li>
             </p>
                 </div>
 

http://git-wip-us.apache.org/repos/asf/hbase-site/blob/48bd3915/hbase-annotations/index.html
----------------------------------------------------------------------
diff --git a/hbase-annotations/index.html b/hbase-annotations/index.html
index 5760b16..4fd067a 100644
--- a/hbase-annotations/index.html
+++ b/hbase-annotations/index.html
@@ -7,7 +7,7 @@
   <head>
     <meta charset="UTF-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <meta name="Date-Revision-yyyymmdd" content="20171216" />
+    <meta name="Date-Revision-yyyymmdd" content="20171217" />
     <meta http-equiv="Content-Language" content="en" />
     <title>Apache HBase - Annotations &#x2013; About</title>
     <link rel="stylesheet" href="./css/apache-maven-fluido-1.5-HBASE.min.css" />
@@ -119,7 +119,7 @@
                         <a href="https://www.apache.org/">The Apache Software Foundation</a>.
             All rights reserved.      
                     
-                  <li id="publishDate" class="pull-right">Last Published: 2017-12-16</li>
+                  <li id="publishDate" class="pull-right">Last Published: 2017-12-17</li>
             </p>
                 </div>
 

http://git-wip-us.apache.org/repos/asf/hbase-site/blob/48bd3915/hbase-annotations/integration.html
----------------------------------------------------------------------
diff --git a/hbase-annotations/integration.html b/hbase-annotations/integration.html
index 7595d64..a04a6d2 100644
--- a/hbase-annotations/integration.html
+++ b/hbase-annotations/integration.html
@@ -7,7 +7,7 @@
   <head>
     <meta charset="UTF-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <meta name="Date-Revision-yyyymmdd" content="20171216" />
+    <meta name="Date-Revision-yyyymmdd" content="20171217" />
     <meta http-equiv="Content-Language" content="en" />
     <title>Apache HBase - Annotations &#x2013; CI Management</title>
     <link rel="stylesheet" href="./css/apache-maven-fluido-1.5-HBASE.min.css" />
@@ -126,7 +126,7 @@
                         <a href="https://www.apache.org/">The Apache Software Foundation</a>.
             All rights reserved.      
                     
-                  <li id="publishDate" class="pull-right">Last Published: 2017-12-16</li>
+                  <li id="publishDate" class="pull-right">Last Published: 2017-12-17</li>
             </p>
                 </div>
 

http://git-wip-us.apache.org/repos/asf/hbase-site/blob/48bd3915/hbase-annotations/issue-tracking.html
----------------------------------------------------------------------
diff --git a/hbase-annotations/issue-tracking.html b/hbase-annotations/issue-tracking.html
index c3265e0..cc9cc58 100644
--- a/hbase-annotations/issue-tracking.html
+++ b/hbase-annotations/issue-tracking.html
@@ -7,7 +7,7 @@
   <head>
     <meta charset="UTF-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <meta name="Date-Revision-yyyymmdd" content="20171216" />
+    <meta name="Date-Revision-yyyymmdd" content="20171217" />
     <meta http-equiv="Content-Language" content="en" />
     <title>Apache HBase - Annotations &#x2013; Issue Management</title>
     <link rel="stylesheet" href="./css/apache-maven-fluido-1.5-HBASE.min.css" />
@@ -123,7 +123,7 @@
                         <a href="https://www.apache.org/">The Apache Software Foundation</a>.
             All rights reserved.      
                     
-                  <li id="publishDate" class="pull-right">Last Published: 2017-12-16</li>
+                  <li id="publishDate" class="pull-right">Last Published: 2017-12-17</li>
             </p>
                 </div>
 

http://git-wip-us.apache.org/repos/asf/hbase-site/blob/48bd3915/hbase-annotations/license.html
----------------------------------------------------------------------
diff --git a/hbase-annotations/license.html b/hbase-annotations/license.html
index ee69c01..a1b3a11 100644
--- a/hbase-annotations/license.html
+++ b/hbase-annotations/license.html
@@ -7,7 +7,7 @@
   <head>
     <meta charset="UTF-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <meta name="Date-Revision-yyyymmdd" content="20171216" />
+    <meta name="Date-Revision-yyyymmdd" content="20171217" />
     <meta http-equiv="Content-Language" content="en" />
     <title>Apache HBase - Annotations &#x2013; Project Licenses</title>
     <link rel="stylesheet" href="./css/apache-maven-fluido-1.5-HBASE.min.css" />
@@ -326,7 +326,7 @@
                         <a href="https://www.apache.org/">The Apache Software Foundation</a>.
             All rights reserved.      
                     
-                  <li id="publishDate" class="pull-right">Last Published: 2017-12-16</li>
+                  <li id="publishDate" class="pull-right">Last Published: 2017-12-17</li>
             </p>
                 </div>
 

http://git-wip-us.apache.org/repos/asf/hbase-site/blob/48bd3915/hbase-annotations/mail-lists.html
----------------------------------------------------------------------
diff --git a/hbase-annotations/mail-lists.html b/hbase-annotations/mail-lists.html
index cba5376..00d7fc4 100644
--- a/hbase-annotations/mail-lists.html
+++ b/hbase-annotations/mail-lists.html
@@ -7,7 +7,7 @@
   <head>
     <meta charset="UTF-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <meta name="Date-Revision-yyyymmdd" content="20171216" />
+    <meta name="Date-Revision-yyyymmdd" content="20171217" />
     <meta http-equiv="Content-Language" content="en" />
     <title>Apache HBase - Annotations &#x2013; Project Mailing Lists</title>
     <link rel="stylesheet" href="./css/apache-maven-fluido-1.5-HBASE.min.css" />
@@ -176,7 +176,7 @@
                         <a href="https://www.apache.org/">The Apache Software Foundation</a>.
             All rights reserved.      
                     
-                  <li id="publishDate" class="pull-right">Last Published: 2017-12-16</li>
+                  <li id="publishDate" class="pull-right">Last Published: 2017-12-17</li>
             </p>
                 </div>
 

http://git-wip-us.apache.org/repos/asf/hbase-site/blob/48bd3915/hbase-annotations/plugin-management.html
----------------------------------------------------------------------
diff --git a/hbase-annotations/plugin-management.html b/hbase-annotations/plugin-management.html
index 84dc077..a967b99 100644
--- a/hbase-annotations/plugin-management.html
+++ b/hbase-annotations/plugin-management.html
@@ -7,7 +7,7 @@
   <head>
     <meta charset="UTF-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <meta name="Date-Revision-yyyymmdd" content="20171216" />
+    <meta name="Date-Revision-yyyymmdd" content="20171217" />
     <meta http-equiv="Content-Language" content="en" />
     <title>Apache HBase - Annotations &#x2013; Project Plugin Management</title>
     <link rel="stylesheet" href="./css/apache-maven-fluido-1.5-HBASE.min.css" />
@@ -271,7 +271,7 @@
                         <a href="https://www.apache.org/">The Apache Software Foundation</a>.
             All rights reserved.      
                     
-                  <li id="publishDate" class="pull-right">Last Published: 2017-12-16</li>
+                  <li id="publishDate" class="pull-right">Last Published: 2017-12-17</li>
             </p>
                 </div>
 

http://git-wip-us.apache.org/repos/asf/hbase-site/blob/48bd3915/hbase-annotations/plugins.html
----------------------------------------------------------------------
diff --git a/hbase-annotations/plugins.html b/hbase-annotations/plugins.html
index e114692..94b111b 100644
--- a/hbase-annotations/plugins.html
+++ b/hbase-annotations/plugins.html
@@ -7,7 +7,7 @@
   <head>
     <meta charset="UTF-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
-    <meta name="Date-Revision-yyyymmdd" content="20171216" />
+    <meta name="Date-Revision-yyyymmdd" content="20171217" />
     <meta http-equiv="Content-Language" content="en" />
     <title>Apache HBase - Annotations &#x2013; Project Plugins</title>
     <link rel="stylesheet" href="./css/apache-maven-fluido-1.5-HBASE.min.css" />
@@ -222,7 +222,7 @@
                         <a href="https://www.apache.org/">The Apache Software Foundation</a>.
             All rights reserved.      
                     
-                  <li id="publishDate" class="pull-right">Last Published: 2017-12-16</li>
+                  <li id="publishDate" class="pull-right">Last Published: 2017-12-17</li>
             </p>
                 </div>