From 56c9783cb32535c6ba065e7127c865c029bc4152 Mon Sep 17 00:00:00 2001
From: kimchy <kimchy@gmail.com>
Date: Tue, 19 Apr 2011 16:16:31 +0300
Subject: [PATCH] remove using field cache when indexing string fields for now

---
 .../mapper/xcontent/StringFieldMapper.java    | 29 +------------------
 1 file changed, 1 insertion(+), 28 deletions(-)

diff --git a/modules/elasticsearch/src/main/java/org/elasticsearch/index/mapper/xcontent/StringFieldMapper.java b/modules/elasticsearch/src/main/java/org/elasticsearch/index/mapper/xcontent/StringFieldMapper.java
index fc55e07e42b..cdf44b93b43 100644
--- a/modules/elasticsearch/src/main/java/org/elasticsearch/index/mapper/xcontent/StringFieldMapper.java
+++ b/modules/elasticsearch/src/main/java/org/elasticsearch/index/mapper/xcontent/StringFieldMapper.java
@@ -93,12 +93,6 @@ public class StringFieldMapper extends AbstractFieldMapper<String> implements In
         public Field field;
     }
 
-    private ThreadLocal<FieldWrapper> fieldCache = new ThreadLocal<FieldWrapper>() {
-        @Override protected FieldWrapper initialValue() {
-            return new FieldWrapper();
-        }
-    };
-
     private String nullValue;
 
     private Boolean includeInAll;
@@ -156,28 +150,7 @@ public class StringFieldMapper extends AbstractFieldMapper<String> implements In
             context.ignoredValue(names.indexName(), value);
             return null;
         }
-        FieldWrapper fieldWrapper = fieldCache.get();
-        Field field = fieldWrapper.field;
-        if (field == null) {
-            field = new Field(names.indexName(), false, value, store, index, termVector);
-        } else {
-            field.setValue(value);
-            fieldWrapper.field = null;
-        }
-        return field;
-    }
-
-    @Override public void processFieldAfterIndex(Fieldable field) {
-        FieldWrapper fieldWrapper = fieldCache.get();
-        if (fieldWrapper.field == null) {
-            Field field1 = (Field) field;
-            field1.setValue("");
-            fieldWrapper.field = field1;
-        }
-    }
-
-    @Override public void close() {
-        fieldCache.remove();
+        return new Field(names.indexName(), false, value, store, index, termVector);
     }
 
     @Override protected String contentType() {