You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@zookeeper.apache.org by ca...@apache.org on 2012/03/17 16:25:57 UTC
svn commit: r1301948 - in /zookeeper/branches/branch-3.4: CHANGES.txt
src/java/main/org/apache/zookeeper/server/DataTree.java
Author: camille
Date: Sat Mar 17 15:25:56 2012
New Revision: 1301948
URL: http://svn.apache.org/viewvc?rev=1301948&view=rev
Log:
ZOOKEEPER-1390. some expensive debug code not protected by a check for debug (breed via camille)
Modified:
zookeeper/branches/branch-3.4/CHANGES.txt
zookeeper/branches/branch-3.4/src/java/main/org/apache/zookeeper/server/DataTree.java
Modified: zookeeper/branches/branch-3.4/CHANGES.txt
URL: http://svn.apache.org/viewvc/zookeeper/branches/branch-3.4/CHANGES.txt?rev=1301948&r1=1301947&r2=1301948&view=diff
==============================================================================
--- zookeeper/branches/branch-3.4/CHANGES.txt (original)
+++ zookeeper/branches/branch-3.4/CHANGES.txt Sat Mar 17 15:25:56 2012
@@ -23,6 +23,8 @@ BUGFIXES:
ZOOKEEPER-1307. zkCli.sh is exiting when an Invalid ACL exception is
thrown from setACL command through client (Kavita Sharma via phunt)
+ ZOOKEEPER-1390. some expensive debug code not protected by a check for debug (breed via camille)
+
IMPROVEMENTS:
ZOOKEEPER-1389. it would be nice if start-foreground used exec $JAVA
Modified: zookeeper/branches/branch-3.4/src/java/main/org/apache/zookeeper/server/DataTree.java
URL: http://svn.apache.org/viewvc/zookeeper/branches/branch-3.4/src/java/main/org/apache/zookeeper/server/DataTree.java?rev=1301948&r1=1301947&r2=1301948&view=diff
==============================================================================
--- zookeeper/branches/branch-3.4/src/java/main/org/apache/zookeeper/server/DataTree.java (original)
+++ zookeeper/branches/branch-3.4/src/java/main/org/apache/zookeeper/server/DataTree.java Sat Mar 17 15:25:56 2012
@@ -777,7 +777,6 @@ public class DataTree {
{
ProcessTxnResult rc = new ProcessTxnResult();
- String debug = "";
try {
rc.clientId = header.getClientId();
rc.cxid = header.getCxid();
@@ -788,7 +787,6 @@ public class DataTree {
switch (header.getType()) {
case OpCode.create:
CreateTxn createTxn = (CreateTxn) txn;
- debug = "Create transaction for " + createTxn.getPath();
rc.path = createTxn.getPath();
createNode(
createTxn.getPath(),
@@ -800,15 +798,11 @@ public class DataTree {
break;
case OpCode.delete:
DeleteTxn deleteTxn = (DeleteTxn) txn;
- debug = "Delete transaction for " + deleteTxn.getPath();
rc.path = deleteTxn.getPath();
deleteNode(deleteTxn.getPath(), header.getZxid());
break;
case OpCode.setData:
SetDataTxn setDataTxn = (SetDataTxn) txn;
- debug = "Set data transaction for "
- + setDataTxn.getPath()
- + " to new value=" + Arrays.toString(setDataTxn.getData());
rc.path = setDataTxn.getPath();
rc.stat = setData(setDataTxn.getPath(), setDataTxn
.getData(), setDataTxn.getVersion(), header
@@ -816,8 +810,6 @@ public class DataTree {
break;
case OpCode.setACL:
SetACLTxn setACLTxn = (SetACLTxn) txn;
- debug = "Set ACL transaction for "
- + setACLTxn.getPath();
rc.path = setACLTxn.getPath();
rc.stat = setACL(setACLTxn.getPath(), setACLTxn.getAcl(),
setACLTxn.getVersion());
@@ -831,16 +823,11 @@ public class DataTree {
break;
case OpCode.check:
CheckVersionTxn checkTxn = (CheckVersionTxn) txn;
- debug = "Check Version transaction for "
- + checkTxn.getPath()
- + " and version="
- + checkTxn.getVersion();
rc.path = checkTxn.getPath();
break;
case OpCode.multi:
MultiTxn multiTxn = (MultiTxn) txn ;
List<Txn> txns = multiTxn.getTxns();
- debug = "Multi transaction with " + txns.size() + " operations";
rc.multiResult = new ArrayList<ProcessTxnResult>();
boolean failed = false;
for (Txn subtxn : txns) {
@@ -902,10 +889,14 @@ public class DataTree {
break;
}
} catch (KeeperException e) {
- LOG.debug("Failed: " + debug, e);
+ if (LOG.isDebugEnabled()) {
+ LOG.debug("Failed: " + header + ":" + txn, e);
+ }
rc.err = e.code().intValue();
} catch (IOException e) {
- LOG.debug("Failed:" + debug, e);
+ if (LOG.isDebugEnabled()) {
+ LOG.debug("Failed: " + header + ":" + txn, e);
+ }
}
/*
* A snapshot might be in progress while we are modifying the data