You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by ch...@apache.org on 2013/09/12 10:17:39 UTC

svn commit: r1522477 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/commit/ConflictValidator.java

Author: chetanm
Date: Thu Sep 12 08:17:39 2013
New Revision: 1522477

URL: http://svn.apache.org/r1522477
Log:
OAK-943 - Provide more details with CommitFailedException

Setting the log level to debug as at times CommitFailedException is expected and such logs create confusion

Modified:
    jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/commit/ConflictValidator.java

Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/commit/ConflictValidator.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/commit/ConflictValidator.java?rev=1522477&r1=1522476&r2=1522477&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/commit/ConflictValidator.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/commit/ConflictValidator.java Thu Sep 12 08:17:39 2013
@@ -85,10 +85,10 @@ public class ConflictValidator extends D
                             CommitFailedException.STATE, 1, "Unresolved conflicts in " + parentAfter.getPath());
 
                     //Conflict details are not made part of ExceptionMessage instead they are
-                    //logged. This to avoid exposing property details to the calller as it might not have
+                    //logged. This to avoid exposing property details to the caller as it might not have
                     //permission to access it
-                    if(log.isWarnEnabled()){
-                        log.warn(getConflictMessage(),ex);
+                    if(log.isDebugEnabled()){
+                        log.debug(getConflictMessage(),ex);
                     }
                     throw ex;
                 }