From 4c1864ddd659b61e0e2c959cf4fb1981e760cee1 Mon Sep 17 00:00:00 2001
From: Michael Bayne <mdb@samskivert.com>
Date: Thu, 5 May 2011 18:56:33 +0000
Subject: [PATCH] This is exactly the sort of place where it's useful to *not*
 turn a complex value into a string and then throw it away when doing debug
 logging.

---
 src/main/java/com/samskivert/depot/PersistenceContext.java | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/main/java/com/samskivert/depot/PersistenceContext.java b/src/main/java/com/samskivert/depot/PersistenceContext.java
index 5e6d496..abf00a9 100644
--- a/src/main/java/com/samskivert/depot/PersistenceContext.java
+++ b/src/main/java/com/samskivert/depot/PersistenceContext.java
@@ -316,7 +316,7 @@ public class PersistenceContext
             log.warning("Cache key must not be null [entry=" + entry + "]", new Exception());
             return;
         }
-        log.debug("storing [key=" + key + ", value=" + entry + "]");
+        log.debug("storing", "key", key, "value", entry);
 
         CacheAdapter.CachedValue<T> element = _cache.lookup(key.getCacheId(), key.getCacheKey());
         T oldEntry = (element != null ? element.getValue() : null);
@@ -328,7 +328,7 @@ public class PersistenceContext
         Set<CacheListener<?>> listeners = _listenerSets.get(key.getCacheId());
         if (listeners != null && listeners.size() > 0) {
             for (CacheListener<?> listener : listeners) {
-                log.debug("cascading [listener=" + listener + "]");
+                log.debug("cascading", "listener", listener);
                 @SuppressWarnings("unchecked")
                     CacheListener<T> casted = (CacheListener<T>)listener;
                 casted.entryCached(entry, oldEntry);
@@ -384,7 +384,7 @@ public class PersistenceContext
                 Set<CacheListener<?>> listeners = _listenerSets.get(cacheId);
                 if (listeners != null && listeners.size() > 0) {
                     for (CacheListener<?> listener : listeners) {
-                        log.debug("cascading [listener=" + listener + "]");
+                        log.debug("cascading", "listener", listener);
                         @SuppressWarnings("unchecked") CacheListener<T> casted =
                             (CacheListener<T>)listener;
                         casted.entryInvalidated(oldEntry);