DATAREDIS-444 - Return result of delete in DefaultHashOperation.

Delete in DefaultHashOperation returns now the result of the HDEL command. Original pull request: spring-projects#184. CLA: 169220160326121428 (Ninad Divadkar)

Author: nsdiv

src/main/java/org/springframework/data/redis/core/BoundHashOperations.java

@@ -52,7 +52,7 @@ public interface BoundHashOperations<H, HK, HV> extends BoundKeyOperations<H> {
 
 Long size();
 
-void delete(Object... keys);
+Long delete(Object... keys);
 
 Map<HK, HV> entries();
 

src/main/java/org/springframework/data/redis/core/DefaultBoundHashOperations.java

@@ -38,15 +38,15 @@ class DefaultBoundHashOperations<H, HK, HV> extends DefaultBoundKeyOperations<H>
  * Constructs a new <code>DefaultBoundHashOperations</code> instance.
  * 
  * @param key
- * @param template
+ * @param operations
  */
 public DefaultBoundHashOperations(H key, RedisOperations<H, ?> operations) {
 super(key, operations);
 this.ops = operations.opsForHash();
 }
 
-public void delete(Object... keys) {
-ops.delete(getKey(), keys);
+public Long delete(Object... keys) {
+return ops.delete(getKey(), keys);
 }
 
 public HV get(Object key) {

src/main/java/org/springframework/data/redis/core/DefaultHashOperations.java

@@ -201,15 +201,14 @@ public List<byte[]> doInRedis(RedisConnection connection) {
 return deserializeHashValues(rawValues);
 }
 
-public void delete(K key, Object... hashKeys) {
+public Long delete(K key, Object... hashKeys) {
 final byte[] rawKey = rawKey(key);
 final byte[][] rawHashKeys = rawHashKeys(hashKeys);
 
-execute(new RedisCallback<Object>() {
+return execute(new RedisCallback<Long>() {
 
-public Object doInRedis(RedisConnection connection) {
-connection.hDel(rawKey, rawHashKeys);
-return null;
+public Long doInRedis(RedisConnection connection) {
+return connection.hDel(rawKey, rawHashKeys);
 }
 }, true);
 }

src/main/java/org/springframework/data/redis/core/HashOperations.java

@@ -28,7 +28,7 @@
  */
 public interface HashOperations<H, HK, HV> {
 
-void delete(H key, Object... hashKeys);
+Long delete(H key, Object... hashKeys);
 
 Boolean hasKey(H key, Object hashKey);
 

src/test/java/org/springframework/data/redis/core/DefaultHashOperationsTests.java

@@ -134,8 +134,9 @@ public void testDelete() {
 HV val2 = hashValueFactory.instance();
 hashOps.put(key, key1, val1);
 hashOps.put(key, key2, val2);
-hashOps.delete(key, key1, key2);
+Long numDeleted = hashOps.delete(key, key1, key2);
 assertTrue(hashOps.keys(key).isEmpty());
+ assertEquals(2L, numDeleted.longValue());
 }
 
 /**