Skip to content

Commit 56269e1

Browse files
committed
fix #993 better fields breaking changes documentation
1 parent 40c63e8 commit 56269e1

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

docs/contents/breaking-changes.md

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,6 +90,7 @@ Prior to Elasticsearch 1.0, you could specify to return only certain fields and
9090
}
9191
...
9292

93+
9394
In many case this could be mapped to the type of DTO you give search (i.e in `.Search<DTO>()`). Elasticsearch 1.0 now always returns the fields as arrays.
9495

9596
...
@@ -99,7 +100,11 @@ In many case this could be mapped to the type of DTO you give search (i.e in `.S
99100
}
100101
...
101102

102-
NEST 1.0 still supports this, but is now a bit more verbose in how it supports mapping the fields back:
103+
Previously, Documents was a collection of T with just the specified fields filled in, other fields would be null of have their default value. Now, Documents is an empty collection.
104+
105+
Previously, DocumentsWithMetaData -> Fields would be an instance of T with just the specified fields filled in, other fields would be null or have their default value. Now, Hits -> Fields is (backed by) a dictionary containing only the specified fields.
106+
107+
NEST 1.0 still supports fields, but is now a bit more verbose in how it supports mapping the fields back:
103108

104109

105110
var fields = _client.Get<DTO>(g => g

0 commit comments

Comments
 (0)