Add GenericProperty.Indexed to support bool? values (#3225)

This commit adds GenericProperty.Indexed to allow a
bool? value to be specified for the field's "index" value,
in line with other properties.

Deprecate the usage of GenericProperty.Index and point users to users
GenericProperty.Indexed. If a user does continue to use GenericProperty.Index
however, set a value of true for FieldIndexOption.Analyzed and
FieldIndexOption.NotAnalyzed, false for FieldIndexOption.No, and null for null.

GenericProperty.Index will be updated to type bool? in 7.x

Fixes #3103
(cherry picked from commit 79caff0)

Author: russcam

src/Nest/Mapping/Types/Specialized/Generic/GenericProperty.cs

@@ -11,9 +11,13 @@ namespace Nest
 	[JsonObject(MemberSerialization.OptIn)]
 	public interface IGenericProperty : IDocValuesProperty
 	{
-		[JsonProperty("index")]
+		[JsonIgnore]
+		[Obsolete("Please use Indexed. Will be fixed in NEST 7.x")]
 		FieldIndexOption? Index { get; set; }
 
+		[JsonProperty("index")]
+		bool? Indexed { get; set; }
+
 		[JsonProperty("term_vector")]
 		TermVectorOption? TermVector { get; set; }
 
@@ -56,6 +60,8 @@ public interface IGenericProperty : IDocValuesProperty
 	[DebuggerDisplay("{DebugDisplay}")]
 	public class GenericProperty : DocValuesPropertyBase, IGenericProperty
 	{
+		private FieldIndexOption? _index;
+
 #pragma warning disable 618
 		public GenericProperty() : base(null) { }
 #pragma warning restore 618
@@ -68,7 +74,31 @@ public GenericProperty() : base(null) { }
 		public int? IgnoreAbove { get; set; }
 		public int? PositionIncrementGap { get; set; }
 		public IStringFielddata Fielddata { get; set; }
-		public FieldIndexOption? Index { get; set; }
+
+		[Obsolete("Please use Indexed. Will be fixed in NEST 7.x")]
+		public FieldIndexOption? Index
+		{
+			get => _index;
+			set
+			{
+				_index = value;
+				switch (_index)
+				{
+					case FieldIndexOption.Analyzed:
+					case FieldIndexOption.NotAnalyzed:
+						Indexed = true;
+						break;
+					case FieldIndexOption.No:
+						Indexed = false;
+						break;
+					default:
+						Indexed = null;
+						break;
+				}
+			}
+		}
+
+		public bool? Indexed { get; set; }
 		public string NullValue { get; set; }
 		public bool? Norms { get; set; }
 		public IndexOptions? IndexOptions { get; set; }
@@ -85,7 +115,31 @@ public class GenericPropertyDescriptor<T>
 		: DocValuesPropertyDescriptorBase<GenericPropertyDescriptor<T>, IGenericProperty, T>, IGenericProperty
 		where T : class
 	{
-		FieldIndexOption? IGenericProperty.Index { get; set; }
+		private FieldIndexOption? _index;
+
+		FieldIndexOption? IGenericProperty.Index
+		{
+			get => _index;
+			set
+			{
+				_index = value;
+				switch (_index)
+				{
+					case FieldIndexOption.Analyzed:
+					case FieldIndexOption.NotAnalyzed:
+						Self.Indexed = true;
+						break;
+					case FieldIndexOption.No:
+						Self.Indexed = false;
+						break;
+					default:
+						Self.Indexed = null;
+						break;
+				}
+			}
+		}
+
+		bool? IGenericProperty.Indexed { get; set; }
 		TermVectorOption? IGenericProperty.TermVector { get; set; }
 		double? IGenericProperty.Boost { get; set; }
 		string IGenericProperty.NullValue { get; set; }
@@ -105,15 +159,19 @@ public GenericPropertyDescriptor() : base(null) { }
 
 		public GenericPropertyDescriptor<T> Type(string type) => Assign(a => a.Type = type);
 
+		[Obsolete("Please use the overload that accepts bool?. Will be fixed in NEST 7.x")]
 		public GenericPropertyDescriptor<T> Index(FieldIndexOption? index = FieldIndexOption.NotAnalyzed) => Assign(a => a.Index = index);
 
+		public GenericPropertyDescriptor<T> Index(bool? index = true) => Assign(a => a.Indexed = index);
+
 		public GenericPropertyDescriptor<T> Boost(double boost) => Assign(a => a.Boost = boost);
 
 		public GenericPropertyDescriptor<T> NullValue(string nullValue) => Assign(a => a.NullValue = nullValue);
 
 		/// <remarks>Removed in 6.x</remarks>
 		public GenericPropertyDescriptor<T> IncludeInAll(bool includeInAll = true) => Assign(a => a.IncludeInAll = includeInAll);
 
+		[Obsolete("Deprecated. Will be removed in NEST 7.x")]
 		public GenericPropertyDescriptor<T> NotAnalyzed() => Index(FieldIndexOption.NotAnalyzed);
 
 		public GenericPropertyDescriptor<T> Index(FieldIndexOption index) => Assign(a => a.Index = index);

src/Tests/Indices/IndexSettings/IndexTemplates/PutIndexTemplate/PutIndexTemplateApiTests.cs

@@ -52,7 +52,7 @@ protected override LazyResponses ClientUsage() => Calls(
 								match_mapping_type = "*",
 								mapping = new
 								{
-									index = "no"
+									index = false
 								}
 							}
 						}
@@ -78,7 +78,7 @@ protected override LazyResponses ClientUsage() => Calls(
 							.MatchMappingType("*")
 							.Mapping(mm => mm
 								.Generic(g => g
-									.Index(FieldIndexOption.No)
+									.Index(false)
 								)
 							)
 						)
@@ -112,7 +112,7 @@ protected override LazyResponses ClientUsage() => Calls(
 									MatchMappingType = "*",
 									Mapping = new GenericProperty
 									{
-										Index = FieldIndexOption.No
+										Indexed = false
 									}
 								}
 							}

src/Tests/Mapping/Types/Specialized/Generic/GenericPropertyTests.cs

@@ -10,18 +10,18 @@ public class GenericPropertyTests : SingleMappingPropertyTestsBase
 		private const string GenericType = "{dynamic_type}";
 		public GenericPropertyTests(WritableCluster cluster, EndpointUsage usage) : base(cluster, usage) { }
 
-		protected override object SingleMappingJson { get; } = new {index = "no", type= GenericType};
+		protected override object SingleMappingJson { get; } = new {index = false, type= GenericType};
 
 		protected override Func<SingleMappingDescriptor<object>, IProperty> FluentSingleMapping => m => m
 			.Generic(g => g
 				.Type(GenericType)
-				.Index(FieldIndexOption.No)
+				.Index(false)
 			);
 
 		protected override IProperty InitializerSingleMapping { get; } = new GenericProperty
 		{
 			Type = GenericType,
-			Index = FieldIndexOption.No
+			Indexed = false
 		};
 	}
 }