Get API: When _source is disabled, the source is still used if fetche…

…d from the transaction log, closes #1927.
elastic · May 8, 2012 · 11064df · 11064df
1 parent ddecd72
commit 11064df
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/src/main/java/org/elasticsearch/index/get/ShardGetService.java b/src/main/java/org/elasticsearch/index/get/ShardGetService.java
@@ -299,9 +299,12 @@ public GetResult innerGet(String type, String id, String[] gFields, boolean real
                                 }
 
                                 FieldMapper<?> x = docMapper.mappers().smartNameFieldMapper(field);
-                                value = searchLookup.source().extractValue(field);
-                                if (x != null && value instanceof String) {
-                                    value = x.valueFromString((String) value);
+                                // only if the field is stored or source is enabled we should add it..
+                                if (docMapper.sourceMapper().enabled() || x == null || x.stored()) {
+                                    value = searchLookup.source().extractValue(field);
+                                    if (x != null && value instanceof String) {
+                                        value = x.valueFromString((String) value);
+                                    }
                                 }
                             }
                         }