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 );
}
}
}