From e2da2114e7b8a3ed18493117657eef7b545c90d3 Mon Sep 17 00:00:00 2001 From: Adrien Grand Date: Tue, 17 Jun 2014 01:50:42 +0200 Subject: [PATCH] Mappings: Allow _version to use `disk` as a doc values format. VersionFieldMapper.defaultDocValuesFormat claims that the default is `disk`. This is not used to choose the DV format in the index but for mappings serialization in order to know when the _version doc values format is different from the default format. This made it impossible to use the `disk` doc values format since mappings would never retain that information at serialization time. Close #6523 --- .../index/mapper/internal/VersionFieldMapper.java | 5 ----- 1 file changed, 5 deletions(-) diff --git a/src/main/java/org/elasticsearch/index/mapper/internal/VersionFieldMapper.java b/src/main/java/org/elasticsearch/index/mapper/internal/VersionFieldMapper.java index 3255ddd9c9d..c6e5ba23fe2 100644 --- a/src/main/java/org/elasticsearch/index/mapper/internal/VersionFieldMapper.java +++ b/src/main/java/org/elasticsearch/index/mapper/internal/VersionFieldMapper.java @@ -102,11 +102,6 @@ public class VersionFieldMapper extends AbstractFieldMapper implements Int super(new Names(NAME, NAME, NAME, NAME), Defaults.BOOST, Defaults.FIELD_TYPE, null, null, null, null, docValuesFormat, null, null, null, ImmutableSettings.EMPTY); } - @Override - protected String defaultDocValuesFormat() { - return "disk"; - } - @Override public void preParse(ParseContext context) throws IOException { super.parse(context);