Unable to set index property on dynamic template for generic type

[tagg7]

NEST/Elasticsearch.Net version: 2.3

Elasticsearch version: 2.3.1

Description of the problem including expected versus actual behavior: There is no way to set a dynamic template to not index all fields by default when setting the mapping.

In Elasticsearch.Net 1.x, it is possible to do the following:

    .DynamicTemplates(d => d
                    .Add(t => t
                        .Name("base")
                        .Match("*")
                        .MatchMappingType("*")
                        .Mapping(m2 => m2
                            .Generic(s => s
                                .Index("no")
                            )
                        )
                    )
                )

However, in Elasticsearch.Net 2.x there is no way to set this. There is no GenericMappingDescriptor available, and the index property is hidden for the "object" type.

[russcam]

Hey @tagg7, thanks for reporting.

@gmarz, @Mpdreamz - was this removed in 2.x for some reason?

Where in NEST 1.x you would have done something like

client.PutTemplate("do_not_index", p => p
    .Template("*")
    .AddMapping<object>(m => m
        .Type("_default_")
        .DynamicTemplates(d => d
            .Add(t => t
                .Name("base")
                .Match("*")
                .MatchMappingType("*")
                .Mapping(m2 => m2
                    .Generic(s => s
                        .Index("no")
                    )
                )
            )
        )
    )
);

You could write an IProperty implementation to do this in 2.x for the time being

public class GenericProperty : PropertyBase
{
    public GenericProperty() : base(null) {}

    [JsonProperty("index")]
    public FieldIndexOption Index { get; set;}
}

client.PutIndexTemplate("do_not_index", p => p
    .Template("*")
    .Mappings(m => m
        .Map("_default_", mm => mm
            .DynamicTemplates(dt => dt
                .DynamicTemplate("base", dtd => dtd
                    .Match("*")
                    .MatchMappingType("*")
                    .Mapping(sm =>
                        new GenericProperty { Index = FieldIndexOption.No }
                    )
                )
            )   
        )
    )
);

which will give the following request

PUT http://localhost:9200/_template/do_not_index
{
  "template": "*",
  "mappings": {
    "_default_": {
      "dynamic_templates": [
        {
          "base": {
            "match": "*",
            "match_mapping_type": "*",
            "mapping": {
              "index": "no"
            }
          }
        }
      ]
    }
  }
}

[tagg7]

Awesome, thank you for the workaround @russcam!

[gmarz]

was this removed in 2.x for some reason?

It was removed, but not for any good reason. Probably missed this during the refactoring. It should be added back.

[russcam]

@tagg7 This will go into the next release of 2.x. Thanks again for raising this 👍