You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by el...@apache.org on 2006/01/12 23:48:42 UTC

svn commit: r368504 - in /directory/trunks/protocol-common/src/main/java/org/apache/protocol/common: AbstractBackingStoreTest.java store/LdifFileLoader.java

Author: elecharny
Date: Thu Jan 12 14:48:38 2006
New Revision: 368504

URL: http://svn.apache.org/viewcvs?rev=368504&view=rev
Log:
Updated the logs
- used the Message Format pattern

Modified:
    directory/trunks/protocol-common/src/main/java/org/apache/protocol/common/AbstractBackingStoreTest.java
    directory/trunks/protocol-common/src/main/java/org/apache/protocol/common/store/LdifFileLoader.java

Modified: directory/trunks/protocol-common/src/main/java/org/apache/protocol/common/AbstractBackingStoreTest.java
URL: http://svn.apache.org/viewcvs/directory/trunks/protocol-common/src/main/java/org/apache/protocol/common/AbstractBackingStoreTest.java?rev=368504&r1=368503&r2=368504&view=diff
==============================================================================
--- directory/trunks/protocol-common/src/main/java/org/apache/protocol/common/AbstractBackingStoreTest.java (original)
+++ directory/trunks/protocol-common/src/main/java/org/apache/protocol/common/AbstractBackingStoreTest.java Thu Jan 12 14:48:38 2006
@@ -318,13 +318,13 @@
                 {
                     ctx.lookup( rdn );
 
-                    log.info( "Found " + rdn + ", will not create." );
+                    log.info( "Found {}, will not create.", rdn );
                 }
                 catch ( Exception e )
                 {
                     ctx.createSubcontext( rdn, attributes );
 
-                    log.info( "Created " + rdn + "." );
+                    log.info( "Created {}.", rdn );
                 }
             }
         }
@@ -366,11 +366,11 @@
 
                 Object stored = ctx.lookup( rdn );
 
-                log.debug( "Lookup for " + rdn + " returned " + stored + "." );
+                log.debug( "Lookup for {} returned {}.", rdn, stored );
 
                 if ( stored == null )
                 {
-                    log.error( rdn + " was null." );
+                    log.error( "{} was null.", rdn );
 
                     throw new IllegalStateException( "LDIF entries not being pushed to disk." );
                 }
@@ -378,7 +378,7 @@
         }
         catch ( Exception e )
         {
-            log.error( "Failed to find " + rdn );
+            log.error( "Failed to find {}", rdn );
 
             if ( log.isDebugEnabled() )
             {

Modified: directory/trunks/protocol-common/src/main/java/org/apache/protocol/common/store/LdifFileLoader.java
URL: http://svn.apache.org/viewcvs/directory/trunks/protocol-common/src/main/java/org/apache/protocol/common/store/LdifFileLoader.java?rev=368504&r1=368503&r2=368504&view=diff
==============================================================================
--- directory/trunks/protocol-common/src/main/java/org/apache/protocol/common/store/LdifFileLoader.java (original)
+++ directory/trunks/protocol-common/src/main/java/org/apache/protocol/common/store/LdifFileLoader.java Thu Jan 12 14:48:38 2006
@@ -161,13 +161,13 @@
                 try
                 {
                     ctx.lookup( rdn );
-                    log.info( "Found " + rdn + ", will not create." );
+                    log.info( "Found {}, will not create.", rdn );
                 }
                 catch ( Exception e )
                 {
                     ctx.createSubcontext( rdn, attributes );
                     count++;
-                    log.info( "Created " + rdn + "." );
+                    log.info( "Created {}.", rdn );
                 }
             }
         }