Search API: Fields listed should exists within a "fields" Json object. Closes #13.

kimchy · kimchy · commit e768f67fa037 · 2010-02-15T10:27:39.000+02:00
diff --git a/modules/elasticsearch/src/main/java/org/elasticsearch/search/internal/InternalSearchHit.java b/modules/elasticsearch/src/main/java/org/elasticsearch/search/internal/InternalSearchHit.java
@@ -120,6 +120,7 @@ public void shard(SearchShardTarget target) {
             builder.raw(source());
         }
         if (fields() != null) {
+            builder.startObject("fields");
             for (SearchHitField field : fields().values()) {
                 if (field.values().isEmpty()) {
                     continue;
@@ -135,6 +136,7 @@ public void shard(SearchShardTarget target) {
                     builder.endArray();
                 }
             }
+            builder.endObject();
         }
         if (explanation() != null) {
             builder.field("_explanation");

Original file line number	Diff line number	Diff line change
`@@ -120,6 +120,7 @@ public void shard(SearchShardTarget target) {`
`120`	`120`	`builder.raw(source());`
`121`	`121`	`}`
`122`	`122`	`if (fields() != null) {`
	`123`	`+ builder.startObject("fields");`
`123`	`124`	`for (SearchHitField field : fields().values()) {`
`124`	`125`	`if (field.values().isEmpty()) {`
`125`	`126`	`continue;`
`@@ -135,6 +136,7 @@ public void shard(SearchShardTarget target) {`
`135`	`136`	`builder.endArray();`
`136`	`137`	`}`
`137`	`138`	`}`
	`139`	`+ builder.endObject();`
`138`	`140`	`}`
`139`	`141`	`if (explanation() != null) {`
`140`	`142`	`builder.field("_explanation");`