From a15b25c9ef617958e2b5b101c81f33ae5767c1bb Mon Sep 17 00:00:00 2001
From: Mark Paluch <mpaluch@pivotal.io>
Date: Tue, 27 Jun 2017 17:01:04 +0200
Subject: [PATCH] DATACMNS-1101 - Cache Value annotation value eagerly.

---
 .../AnnotationBasedPersistentProperty.java      | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/src/main/java/org/springframework/data/mapping/model/AnnotationBasedPersistentProperty.java b/src/main/java/org/springframework/data/mapping/model/AnnotationBasedPersistentProperty.java
index 03702b56c..061cc288f 100644
--- a/src/main/java/org/springframework/data/mapping/model/AnnotationBasedPersistentProperty.java
+++ b/src/main/java/org/springframework/data/mapping/model/AnnotationBasedPersistentProperty.java
@@ -52,7 +52,7 @@ public abstract class AnnotationBasedPersistentProperty<P extends PersistentProp
 
 	private static final String SPRING_DATA_PACKAGE = "org.springframework.data";
 
-	private final Value value;
+	private final String value;
 	private final Map<Class<? extends Annotation>, Optional<? extends Annotation>> annotationCache = new HashMap<>();
 
 	private final Lazy<Boolean> usePropertyAccess = Lazy.of(() -> {
@@ -78,7 +78,13 @@ public abstract class AnnotationBasedPersistentProperty<P extends PersistentProp
 
 		populateAnnotationCache(property);
 
-		this.value = findAnnotation(Value.class);
+		Value value = findAnnotation(Value.class);
+
+		if (value != null) {
+			this.value = value.value();
+		} else {
+			this.value = null;
+		}
 	}
 
 	/**
@@ -152,12 +158,7 @@ public abstract class AnnotationBasedPersistentProperty<P extends PersistentProp
 	 */
 	@Override
 	public String getSpelExpression() {
-
-		if (value != null) {
-			return value.value();
-		}
-
-		return null;
+		return value;
 	}
 
 	/**