honeymoose/OpenSearch
1
0
Fork 0
mirror of https://github.com/honeymoose/OpenSearch.git synced 2025-02-17 10:25:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

CommitStats doesn't need to allow for null values in commit user data

Browse Source 
Lucene forbids writing those and MapBuilder.immutableMap doesn't like them either, as discovered by @brwe

Closes #10774
This commit is contained in:
Boaz Leskes 2015-04-24 12:46:13 +02:00
parent 54cf885d14
commit 389ce39919
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/main/java/org/elasticsearch/index/engine/CommitStats.java
View File

@ -78,7 +78,7 @@ public final class CommitStats implements Streamable, ToXContent {
public void readFrom(StreamInput in) throws IOException {
MapBuilder<String, String> builder = MapBuilder.newMapBuilder();
for (int i = in.readVInt(); i > 0; i--) {
builder.put(in.readString(), in.readOptionalString());
builder.put(in.readString(), in.readString());
}
userData = builder.immutableMap();
generation = in.readLong();
@ -90,7 +90,7 @@ public final class CommitStats implements Streamable, ToXContent {
out.writeVInt(userData.size());
for (Map.Entry<String, String> entry : userData.entrySet()) {
out.writeString(entry.getKey());
out.writeOptionalString(entry.getValue());
out.writeString(entry.getValue());
}
out.writeLong(generation);
out.writeString(id);