Skip to content

Commit 2cc3055

Browse files
odrotbohmchristophstrobl
authored andcommitted
DATAREDIS-425 - Adapted to changes in Spring Data KeyValue APIs.
We now favor shadowing fields over using protected accessor methods. Original Pull Request: #156
1 parent 92274a8 commit 2cc3055

File tree

2 files changed

+16
-6
lines changed

2 files changed

+16
-6
lines changed

src/main/java/org/springframework/data/redis/core/mapping/RedisMappingContext.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,13 +51,15 @@
5151
* Redis specific {@link MappingContext}.
5252
*
5353
* @author Christoph Strobl
54+
* @author Oliver Gierke
5455
* @since 1.7
5556
*/
5657
public class RedisMappingContext extends KeyValueMappingContext {
5758

58-
private MappingConfiguration mappingConfiguration;
59+
private final MappingConfiguration mappingConfiguration;
60+
private final TimeToLiveAccessor timeToLiveAccessor;
61+
5962
private KeySpaceResolver fallbackKeySpaceResolver;
60-
private TimeToLiveAccessor timeToLiveAccessor;
6163

6264
/**
6365
* Creates new {@link RedisMappingContext} with empty {@link MappingConfiguration}.

src/main/java/org/springframework/data/redis/repository/support/RedisRepositoryFactory.java

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright 2015 the original author or authors.
2+
* Copyright 2015-2016 the original author or authors.
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
55
* you may not use this file except in compliance with the License.
@@ -42,16 +42,20 @@
4242
* {@link org.springframework.data.keyvalue.repository.KeyValueRepository}.
4343
*
4444
* @author Christoph Strobl
45+
* @author Oliver Gierke
4546
* @since 1.7
4647
*/
4748
public class RedisRepositoryFactory extends KeyValueRepositoryFactory {
4849

50+
private final KeyValueOperations operations;
51+
private final Class<? extends AbstractQueryCreator<?, ?>> queryCreator;
52+
4953
/**
5054
* @param keyValueOperations
5155
* @see KeyValueRepositoryFactory#KeyValueRepositoryFactory(KeyValueOperations)
5256
*/
5357
public RedisRepositoryFactory(KeyValueOperations keyValueOperations) {
54-
super(keyValueOperations);
58+
this(keyValueOperations, DEFAULT_QUERY_CREATOR);
5559
}
5660

5761
/**
@@ -62,6 +66,9 @@ public RedisRepositoryFactory(KeyValueOperations keyValueOperations) {
6266
public RedisRepositoryFactory(KeyValueOperations keyValueOperations,
6367
Class<? extends AbstractQueryCreator<?, ?>> queryCreator) {
6468
super(keyValueOperations, queryCreator);
69+
70+
this.operations = keyValueOperations;
71+
this.queryCreator = queryCreator;
6572
}
6673

6774
/*
@@ -70,7 +77,7 @@ public RedisRepositoryFactory(KeyValueOperations keyValueOperations,
7077
*/
7178
@Override
7279
protected QueryLookupStrategy getQueryLookupStrategy(Key key, EvaluationContextProvider evaluationContextProvider) {
73-
return new RedisQueryLookupStrategy(key, evaluationContextProvider, getKeyValueOperations(), getQueryCreator());
80+
return new RedisQueryLookupStrategy(key, evaluationContextProvider, operations, queryCreator);
7481
}
7582

7683
/*
@@ -81,7 +88,8 @@ protected QueryLookupStrategy getQueryLookupStrategy(Key key, EvaluationContextP
8188
@SuppressWarnings("unchecked")
8289
public <T, ID extends Serializable> EntityInformation<T, ID> getEntityInformation(Class<T> domainClass) {
8390

84-
RedisPersistentEntity<T> entity = (RedisPersistentEntity<T>) getMappingContext().getPersistentEntity(domainClass);
91+
RedisPersistentEntity<T> entity = (RedisPersistentEntity<T>) operations.getMappingContext()
92+
.getPersistentEntity(domainClass);
8593
EntityInformation<T, ID> entityInformation = (EntityInformation<T, ID>) new MappingRedisEntityInformation<T, ID>(
8694
entity);
8795

0 commit comments

Comments
 (0)