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

Matteo Bertozzi commented on HBASE-16598:
-----------------------------------------

this does not seem to compile, since we removed ZOOKEEPER_USEMULTI from 
HConstants.java
{code}
   public static void multiOrSequential(ZooKeeperWatcher zkw, List<ZKUtilOp> 
ops,
       boolean runSequentialOnMultiFailure) throws KeeperException {
+    if (zkw.getConfiguration().get(HConstants.ZOOKEEPER_USEMULTI) != null) {
+      LOG.warn(HConstants.ZOOKEEPER_USEMULTI + " is deprecated. Default to 
true always.");
+    }
{code}

but other than that, patch looks great

> Clean up zookeeper useMulti in HBase code
> -----------------------------------------
>
>                 Key: HBASE-16598
>                 URL: https://issues.apache.org/jira/browse/HBASE-16598
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Jerry He
>            Assignee: Jerry He
>         Attachments: HBASE-16598.patch
>
>
> We have zookeeper useMulti default true since HBase 1.0.0
> And in our Doc, "ZooKeeper 3.4.x is required as of HBase 1.0.0"
> The benefit of useMulti is obvious. 
> Let's clean up the code to remove useMulti false case so that we don't have 
> to continue with the hassle of maintaining two version of the code (e.g. in 
> replication) .  No go back to pre 3.4.x Zookeeper.



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

Reply via email to