Spring/spring-data-redis
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix Jedis persist does not execute as part of a transaction

Browse Source 
DATAREDIS-160
This commit is contained in:
Jennifer Hickey
2013-04-01 16:44:51 -07:00
parent 634e8ccd72
commit 4e13ac5b86
2 changed files with 1 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/main/java/org/springframework/data/redis/connection/jedis/JedisConnection.java
View File

@@ -699,7 +699,7 @@ public class JedisConnection implements RedisConnection {
public Boolean persist(byte[] key) {
try {
if (isQueueing()) {
client.persist(key);
transaction.persist(key);
return null;
}
if (isPipelined()) {

8
src/test/java/org/springframework/data/redis/connection/jedis/JedisConnectionTransactionIntegrationTests.java
View File

@@ -1,8 +1,5 @@
package org.springframework.data.redis.connection.jedis;
import static org.junit.Assert.fail;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.List;
@@ -10,7 +7,6 @@ import org.junit.Ignore;
import org.junit.Test;
import org.junit.runner.RunWith;
import org.springframework.data.redis.RedisSystemException;
import org.springframework.data.redis.connection.RedisPipelineException;
import org.springframework.test.context.ContextConfiguration;
import org.springframework.test.context.junit4.SpringJUnit4ClassRunner;
@@ -19,10 +15,6 @@ import org.springframework.test.context.junit4.SpringJUnit4ClassRunner;
public class JedisConnectionTransactionIntegrationTests extends
JedisConnectionPipelineIntegrationTests {
@Ignore("DATAREDIS-160 Jedis persist does not execute as part of a transaction")
public void testPersist() throws Exception {
}
@Ignore("DATAREDIS-159 brPop executes twice in a transaction")
public void testBRPop() {
}