Support sub-aggs for single bucket aggregations (#6633) (#6634)

* Aggregates implement IAggregate

* Response to schema changes

* Remove Aggregate class

* Fix single bucket aggregate types

Co-authored-by: Steve Gordon <sgordon@hotmail.co.uk>

Author: github-actions[bot]

src/Elastic.Clients.Elasticsearch/Common/Aggregations/Aggregate.cs renamed to src/Elastic.Clients.Elasticsearch/Common/Aggregations/IAggregate.cs

@@ -5,12 +5,3 @@
 namespace Elastic.Clients.Elasticsearch.Aggregations;
 
 public interface IAggregate { }
-
-/// <summary>
-/// Base class for all aggregates.
-/// </summary>
-public abstract class Aggregate : IAggregate
-{
-}
-
-

src/Elastic.Clients.Elasticsearch/_Generated/Api/Snapshot/GetSnapshotRequest.g.cs

@@ -39,6 +39,9 @@ public sealed class GetSnapshotRequestParameters : RequestParameters<GetSnapshot
 		[JsonIgnore]
 		public bool? IndexDetails { get => Q<bool?>("index_details"); set => Q("index_details", value); }
 
+		[JsonIgnore]
+		public bool? IndexNames { get => Q<bool?>("index_names"); set => Q("index_names", value); }
+
 		[JsonIgnore]
 		public bool? IncludeRepository { get => Q<bool?>("include_repository"); set => Q("include_repository", value); }
 
@@ -85,6 +88,9 @@ public GetSnapshotRequest(Elastic.Clients.Elasticsearch.Name repository, Elastic
 		[JsonIgnore]
 		public bool? IndexDetails { get => Q<bool?>("index_details"); set => Q("index_details", value); }
 
+		[JsonIgnore]
+		public bool? IndexNames { get => Q<bool?>("index_names"); set => Q("index_names", value); }
+
 		[JsonIgnore]
 		public bool? IncludeRepository { get => Q<bool?>("include_repository"); set => Q("include_repository", value); }
 
@@ -129,6 +135,7 @@ internal GetSnapshotRequestDescriptor()
 		public GetSnapshotRequestDescriptor IgnoreUnavailable(bool? ignoreUnavailable = true) => Qs("ignore_unavailable", ignoreUnavailable);
 		public GetSnapshotRequestDescriptor IncludeRepository(bool? includeRepository = true) => Qs("include_repository", includeRepository);
 		public GetSnapshotRequestDescriptor IndexDetails(bool? indexDetails = true) => Qs("index_details", indexDetails);
+		public GetSnapshotRequestDescriptor IndexNames(bool? indexNames = true) => Qs("index_names", indexNames);
 		public GetSnapshotRequestDescriptor MasterTimeout(Elastic.Clients.Elasticsearch.Duration? masterTimeout) => Qs("master_timeout", masterTimeout);
 		public GetSnapshotRequestDescriptor Offset(int? offset) => Qs("offset", offset);
 		public GetSnapshotRequestDescriptor Order(Elastic.Clients.Elasticsearch.SortOrder? order) => Qs("order", order);

src/Elastic.Clients.Elasticsearch/_Generated/Types/Aggregations/AdjacencyMatrixAggregate.g.cs

@@ -24,7 +24,7 @@
 #nullable restore
 namespace Elastic.Clients.Elasticsearch.Aggregations
 {
-	public sealed partial class AdjacencyMatrixAggregate : Aggregate
+	public sealed partial class AdjacencyMatrixAggregate : IAggregate
 	{
 		[JsonInclude]
 		[JsonPropertyName("buckets")]

src/Elastic.Clients.Elasticsearch/_Generated/Types/Aggregations/AutoDateHistogramAggregate.g.cs

@@ -24,7 +24,7 @@
 #nullable restore
 namespace Elastic.Clients.Elasticsearch.Aggregations
 {
-	public sealed partial class AutoDateHistogramAggregate : Aggregate
+	public sealed partial class AutoDateHistogramAggregate : IAggregate
 	{
 		[JsonInclude]
 		[JsonPropertyName("buckets")]

src/Elastic.Clients.Elasticsearch/_Generated/Types/Aggregations/AvgAggregate.g.cs

@@ -24,7 +24,7 @@
 #nullable restore
 namespace Elastic.Clients.Elasticsearch.Aggregations
 {
-	public sealed partial class AvgAggregate : Aggregate
+	public sealed partial class AvgAggregate : IAggregate
 	{
 		[JsonInclude]
 		[JsonPropertyName("meta")]

src/Elastic.Clients.Elasticsearch/_Generated/Types/Aggregations/BoxPlotAggregate.g.cs

@@ -24,7 +24,7 @@
 #nullable restore
 namespace Elastic.Clients.Elasticsearch.Aggregations
 {
-	public sealed partial class BoxPlotAggregate : Aggregate
+	public sealed partial class BoxPlotAggregate : IAggregate
 	{
 		[JsonInclude]
 		[JsonPropertyName("lower")]

src/Elastic.Clients.Elasticsearch/_Generated/Types/Aggregations/CardinalityAggregate.g.cs

@@ -24,7 +24,7 @@
 #nullable restore
 namespace Elastic.Clients.Elasticsearch.Aggregations
 {
-	public sealed partial class CardinalityAggregate : Aggregate
+	public sealed partial class CardinalityAggregate : IAggregate
 	{
 		[JsonInclude]
 		[JsonPropertyName("meta")]

src/Elastic.Clients.Elasticsearch/_Generated/Types/Aggregations/ChildrenAggregate.g.cs

@@ -24,11 +24,12 @@
 #nullable restore
 namespace Elastic.Clients.Elasticsearch.Aggregations
 {
-	public sealed partial class ChildrenAggregate : Aggregate
+	[JsonConverter(typeof(ChildrenAggregateConverter))]
+	public sealed partial class ChildrenAggregate : AggregateDictionary, IAggregate
 	{
-		[JsonInclude]
-		[JsonPropertyName("aggregations")]
-		public Elastic.Clients.Elasticsearch.Aggregations.AggregateDictionary Aggregations { get; init; }
+		public ChildrenAggregate(IReadOnlyDictionary<string, IAggregate> backingDictionary) : base(backingDictionary)
+		{
+		}
 
 		[JsonInclude]
 		[JsonPropertyName("doc_count")]
@@ -38,4 +39,49 @@ public sealed partial class ChildrenAggregate : Aggregate
 		[JsonPropertyName("meta")]
 		public Dictionary<string, object>? Meta { get; init; }
 	}
+
+	internal sealed class ChildrenAggregateConverter : JsonConverter<ChildrenAggregate>
+	{
+		public override ChildrenAggregate? Read(ref Utf8JsonReader reader, Type typeToConvert, JsonSerializerOptions options)
+		{
+			if (reader.TokenType != JsonTokenType.StartObject)
+				throw new JsonException($"Expected {JsonTokenType.StartObject} but read {reader.TokenType}.");
+			var subAggs = new Dictionary<string, IAggregate>(); // TODO - Optimise this and only create if we need it.
+			long docCount = default;
+			Dictionary<string, object>? meta = default;
+			while (reader.Read())
+			{
+				if (reader.TokenType == JsonTokenType.EndObject)
+					break;
+				if (reader.TokenType != JsonTokenType.PropertyName)
+					throw new JsonException($"Expected {JsonTokenType.PropertyName} but read {reader.TokenType}.");
+				var name = reader.GetString(); // TODO: Future optimisation, get raw bytes span and parse based on those
+				reader.Read();
+				if (name.Equals("doc_count", StringComparison.Ordinal))
+				{
+					docCount = JsonSerializer.Deserialize<long>(ref reader, options);
+					continue;
+				}
+
+				if (name.Equals("meta", StringComparison.Ordinal))
+				{
+					meta = JsonSerializer.Deserialize<Dictionary<string, object>?>(ref reader, options);
+					continue;
+				}
+
+				if (name.Contains("#"))
+				{
+					AggregateDictionaryConverter.ReadAggregate(ref reader, options, subAggs, name);
+					continue;
+				}
+
+				throw new JsonException("Unknown property read from JSON.");
+			}
+
+			return new ChildrenAggregate(subAggs)
+			{ DocCount = docCount, Meta = meta };
+		}
+
+		public override void Write(Utf8JsonWriter writer, ChildrenAggregate value, JsonSerializerOptions options) => throw new NotImplementedException();
+	}
 }

src/Elastic.Clients.Elasticsearch/_Generated/Types/Aggregations/CompositeAggregate.g.cs

@@ -24,7 +24,7 @@
 #nullable restore
 namespace Elastic.Clients.Elasticsearch.Aggregations
 {
-	public sealed partial class CompositeAggregate : Aggregate
+	public sealed partial class CompositeAggregate : IAggregate
 	{
 		[JsonInclude]
 		[JsonPropertyName("after_key")]

src/Elastic.Clients.Elasticsearch/_Generated/Types/Aggregations/CumulativeCardinalityAggregate.g.cs

@@ -24,7 +24,7 @@
 #nullable restore
 namespace Elastic.Clients.Elasticsearch.Aggregations
 {
-	public sealed partial class CumulativeCardinalityAggregate : Aggregate
+	public sealed partial class CumulativeCardinalityAggregate : IAggregate
 	{
 		[JsonInclude]
 		[JsonPropertyName("meta")]