Ensure Sha1KeyStrategy is the default.

Missed that HashCodeKeyStrategy was initialized as the default with in the MemcachedCache instance.

Author: raykrueger

src/main/java/com/googlecode/hibernate/memcached/MemcachedCache.java

@@ -54,7 +54,7 @@ public class MemcachedCache implements Cache {
     private final String clearIndexKey;
     private int cacheTimeSeconds = 300;
     private boolean clearSupported = false;
-    private KeyStrategy keyStrategy = new HashCodeKeyStrategy();
+    private KeyStrategy keyStrategy = new Sha1KeyStrategy();
     private boolean dogpilePreventionEnabled = false;
     private double dogpilePreventionExpirationFactor = 2;
 

src/main/java/com/googlecode/hibernate/memcached/MemcachedCacheProvider.java

@@ -37,7 +37,7 @@
  * <td>Space delimited list of memcached instances in host:port format</td></tr>
  * <tr><td>hibernate.memcached.cacheTimeSeconds</td><td>300</td>
  * <td>The default number of seconds items should be cached. Can be overriden at the regon level.</td></tr>
- * <tr><td>hibernate.memcached.keyStrategy</td><td>{@link HashCodeKeyStrategy}</td>
+ * <tr><td>hibernate.memcached.keyStrategy</td><td>{@link Sha1KeyStrategy}</td>
  * <td>Sets the strategy class to to use for generating cache keys.
  * Must provide a class name that implements {@link com.googlecode.hibernate.memcached.KeyStrategy}</td></tr>
  * <tr><td>hibernate.memcached.readBufferSize</td>
@@ -199,4 +199,4 @@ public void stop() {
     public boolean isMinimalPutsEnabledByDefault() {
         return true;
     }
-}
+}

src/test/groovy/com/googlecode/hibernate/memcached/MemcachedCacheTest.groovy

@@ -25,6 +25,7 @@ class MemcachedCacheTest extends BaseTestCase {
     void test_dogpile_cache_miss() {
         MockMemcached mockCache = new MockMemcached()
         cache = new MemcachedCache("region", mockCache)
+        cache.setKeyStrategy(new HashCodeKeyStrategy());
         cache.dogpilePreventionEnabled = true
         cache.cacheTimeSeconds = 1
         cache.dogpilePreventionExpirationFactor = 2
@@ -39,6 +40,7 @@ class MemcachedCacheTest extends BaseTestCase {
     void test_dogpile_cache_hit() {
         MockMemcached mockCache = new MockMemcached()
         cache = new MemcachedCache("region", mockCache)
+        cache.setKeyStrategy(new HashCodeKeyStrategy());
         cache.dogpilePreventionEnabled = true
         cache.cacheTimeSeconds = 1
         cache.dogpilePreventionExpirationFactor = 2
@@ -48,4 +50,4 @@ class MemcachedCacheTest extends BaseTestCase {
         assertEquals MemcachedCache.DOGPILE_TOKEN, mockCache.cache["region:0:3556498.dogpileTokenKey"]
     }
 
-}
+}