svn commit: r1782973 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/UserImporter.java

[angela]

Author: angela
Date: Tue Feb 14 15:16:28 2017
New Revision: 1782973

URL: http://svn.apache.org/viewvc?rev=1782973&view=rev
Log:
OAK-5654 : Improve log output with UserImporter

Modified:
    
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/UserImporter.java

Modified: 
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/UserImporter.java
URL: 
http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/UserImporter.java?rev=1782973&r1=1782972&r2=1782973&view=diff
==============================================================================
--- 
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/UserImporter.java
 (original)
+++ 
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/user/UserImporter.java
 Tue Feb 14 15:16:28 2017
@@ -233,7 +233,7 @@ class UserImporter implements ProtectedP
         } else {
             Authorizable a = userManager.getAuthorizable(parent);
             if (a == null) {
-                log.warn("Cannot handle protected PropInfo " + propInfo + ". 
Node " + parent + " doesn't represent a valid Authorizable.");
+                log.debug("Cannot handle protected PropInfo " + propInfo + ". 
Node " + parent + " doesn't represent an Authorizable.");
                 return false;
             }
 
@@ -618,7 +618,7 @@ class UserImporter implements ProtectedP
                 } else {
                     handleFailure("New member of " + gr + ": No such 
authorizable (NodeID = " + memberContentId + ')');
                     if (importBehavior == ImportBehavior.BESTEFFORT) {
-                        log.info("ImportBehavior.BESTEFFORT: Remember 
non-existing member for processing.");
+                        log.debug("ImportBehavior.BESTEFFORT: Remember 
non-existing member for processing.");
                         /* since we ignore the set of failed ids later on and
                            don't know the real memberId => use fake memberId as
                            value in the map */
@@ -644,7 +644,7 @@ class UserImporter implements ProtectedP
 
             // handling non-existing members in case of best-effort
             if (!nonExisting.isEmpty()) {
-                log.info("ImportBehavior.BESTEFFORT: Found " + 
nonExisting.size() + " entries of rep:members pointing to non-existing 
authorizables. Adding to rep:members.");
+                log.debug("ImportBehavior.BESTEFFORT: Found " + 
nonExisting.size() + " entries of rep:members pointing to non-existing 
authorizables. Adding to rep:members.");
                 Tree groupTree = root.getTree(gr.getPath());
 
                 MembershipProvider membershipProvider = 
userManager.getMembershipProvider();
@@ -716,7 +716,7 @@ class UserImporter implements ProtectedP
                     handleFailure("Failed to grant impersonation for " + 
principalName + " on " + a);
                     if (importBehavior == ImportBehavior.BESTEFFORT &&
                             getPrincipalManager().getPrincipal(principalName) 
== null) {
-                        log.info("ImportBehavior.BESTEFFORT: Remember 
non-existing impersonator for special processing.");
+                        log.debug("ImportBehavior.BESTEFFORT: Remember 
non-existing impersonator for special processing.");
                         nonExisting.add(principalName);
                     }
                 }