[infinispan-commits] Infinispan SVN: r511 - trunk/core/src/main/java/org/infinispan/loaders/decorators.

[infinispan-commits at lists.jboss.org]

Author: galder.zamarreno at jboss.com
Date: 2009-07-02 04:56:40 -0400 (Thu, 02 Jul 2009)
New Revision: 511

Modified:
   trunk/core/src/main/java/org/infinispan/loaders/decorators/ReadOnlyStore.java
Log:
Added TRACE log messages so that it's more obvious when a read only cache store is in use.

Modified: trunk/core/src/main/java/org/infinispan/loaders/decorators/ReadOnlyStore.java
===================================================================
--- trunk/core/src/main/java/org/infinispan/loaders/decorators/ReadOnlyStore.java	2009-07-01 12:49:55 UTC (rev 510)
+++ trunk/core/src/main/java/org/infinispan/loaders/decorators/ReadOnlyStore.java	2009-07-02 08:56:40 UTC (rev 511)
@@ -4,6 +4,8 @@
 import org.infinispan.loaders.CacheStore;
 import org.infinispan.loaders.modifications.Modification;
 import org.infinispan.transaction.xa.GlobalTransaction;
+import org.infinispan.util.logging.Log;
+import org.infinispan.util.logging.LogFactory;
 
 import java.io.ObjectInput;
 import java.util.List;
@@ -16,6 +18,7 @@
  * @since 4.0
  */
 public class ReadOnlyStore extends AbstractDelegatingStore {
+   private static final Log log = LogFactory.getLog(ReadOnlyStore.class);
 
    public ReadOnlyStore(CacheStore delegate) {
       super(delegate);
@@ -23,41 +26,42 @@
 
    @Override
    public void store(InternalCacheEntry ed) {
-      // no-op
+      if (log.isTraceEnabled()) log.trace("Ignoring store invocation"); 
    }
 
    @Override
    public void fromStream(ObjectInput inputStream) {
-      // no-op
+      if (log.isTraceEnabled()) log.trace("Ignoring writing contents of stream to store");
    }
 
    @Override
    public void clear() {
-      // no-op
+      if (log.isTraceEnabled()) log.trace("Ignoring clear invocation");
    }
 
    @Override
    public boolean remove(Object key) {
+      if (log.isTraceEnabled()) log.trace("Ignoring removal of key");
       return false;  // no-op
    }
 
    @Override
    public void purgeExpired() {
-      // no-op
+      if (log.isTraceEnabled()) log.trace("Ignoring purge expired invocation");
    }
 
    @Override
    public void commit(GlobalTransaction tx) {
-      // no-op
+      if (log.isTraceEnabled()) log.trace("Ignoring transactional commit call");
    }
 
    @Override
    public void rollback(GlobalTransaction tx) {
-      // no-op
+      if (log.isTraceEnabled()) log.trace("Ignoring transactional rollback call");
    }
 
    @Override
    public void prepare(List<? extends Modification> list, GlobalTransaction tx, boolean isOnePhase) {
-      // no-op
+      if (log.isTraceEnabled()) log.trace("Ignoring transactional prepare call");
    }
 }