zookeeper-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From maha...@apache.org
Subject svn commit: r1240949 - in /zookeeper/trunk: CHANGES.txt src/java/main/org/apache/zookeeper/server/DataTree.java src/java/main/org/apache/zookeeper/server/PrepRequestProcessor.java
Date Mon, 06 Feb 2012 09:39:16 GMT
Author: mahadev
Date: Mon Feb  6 09:39:15 2012
New Revision: 1240949

URL: http://svn.apache.org/viewvc?rev=1240949&view=rev
Log:
ZOOKEEPER-1340. multi problem - typical user operations are generating ERROR level messages
in the server (phunt via mahadev)

Modified:
    zookeeper/trunk/CHANGES.txt
    zookeeper/trunk/src/java/main/org/apache/zookeeper/server/DataTree.java
    zookeeper/trunk/src/java/main/org/apache/zookeeper/server/PrepRequestProcessor.java

Modified: zookeeper/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/zookeeper/trunk/CHANGES.txt?rev=1240949&r1=1240948&r2=1240949&view=diff
==============================================================================
--- zookeeper/trunk/CHANGES.txt (original)
+++ zookeeper/trunk/CHANGES.txt Mon Feb  6 09:39:15 2012
@@ -121,6 +121,9 @@ BUGFIXES:
   to exist. (phunt via mahadev)
 
   ZOOKEEPER-1327. there are still remnants of hadoop urls. (Harsh J via mahadev)
+
+  ZOOKEEPER-1340. multi problem - typical user operations are generating ERROR level 
+  messages in the server (phunt via mahadev)
  
 IMPROVEMENTS:
 

Modified: zookeeper/trunk/src/java/main/org/apache/zookeeper/server/DataTree.java
URL: http://svn.apache.org/viewvc/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/DataTree.java?rev=1240949&r1=1240948&r2=1240949&view=diff
==============================================================================
--- zookeeper/trunk/src/java/main/org/apache/zookeeper/server/DataTree.java (original)
+++ zookeeper/trunk/src/java/main/org/apache/zookeeper/server/DataTree.java Mon Feb  6 09:39:15
2012
@@ -849,10 +849,10 @@ public class DataTree {
                     break;
             }
         } catch (KeeperException e) {
-             LOG.warn("Failed: " + debug, e);
-             rc.err = e.code().intValue();
+            LOG.debug("Failed: " + debug, e);
+            rc.err = e.code().intValue();
         } catch (IOException e) {
-            LOG.warn("Failed:" + debug, e);
+            LOG.debug("Failed:" + debug, e);
         }
         /*
          * A snapshot might be in progress while we are modifying the data

Modified: zookeeper/trunk/src/java/main/org/apache/zookeeper/server/PrepRequestProcessor.java
URL: http://svn.apache.org/viewvc/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/PrepRequestProcessor.java?rev=1240949&r1=1240948&r2=1240949&view=diff
==============================================================================
--- zookeeper/trunk/src/java/main/org/apache/zookeeper/server/PrepRequestProcessor.java (original)
+++ zookeeper/trunk/src/java/main/org/apache/zookeeper/server/PrepRequestProcessor.java Mon
Feb  6 09:39:15 2012
@@ -542,11 +542,12 @@ public class PrepRequestProcessor extend
                             ke = e;
                             type = OpCode.error;
                             txn = new ErrorTxn(e.code().intValue());
-                            LOG.error(">>>> Got user-level KeeperException when
processing "
-                                    + request.toString()
+
+                            LOG.info("Got user-level KeeperException when processing "
+                                    + request.toString() + " aborting remaining multi ops."
                                     + " Error Path:" + e.getPath()
                                     + " Error:" + e.getMessage());
-                            LOG.error(">>>> ABORTING remaing MultiOp ops");
+
                             request.setException(e);
 
                             /* Rollback change records from failed multi-op */



Mime
View raw message