Missing && operator between aggregations about elasticsearch-net HOT 5 CLOSED

I see. OK. Thanks.

from elasticsearch-net.

Hi @gpetrou! With the new generator design, aggregation variants do not derive from a common base-class anymore (the same thing as for the Query variants).

I'm fine with adding back this functionality to Aggregation, but this will still require to wrap the individual aggregation variants into the container first, before using the operator.

var aggregations = Aggregation.Min(...) && Aggregation.Max(...);

would work fine in this case, but the previously available syntax won't:

var aggregations = new MinAggregation{...} && new MaxAggregation{...};

from elasticsearch-net.

Well, nevermind. I remember why I did not re-implement this 😋

In the versions prior to 8.12, the name of the aggregation was pulled into the actual variant. For example, MinAggregation has a constructor that accepts the aggregation name.

In the versions starting with 8.13, this got changed to be consistent with the resulting JSON structure (the aggregation name is a key in a dictionary and the aggregation variant is the corresponding value).

The old design caused many issues, because it required a lot of hacks during serialization (have a look at AggregationDictionary).

In consequence, this means that we can not compose a dictionary of aggregations from the individual aggregation variants using the && operator, because there is no way to "store" the aggregation name.

But in my opinion, the reason for introducing && in NEST, is no longer relevant as well. In NEST, aggregation dictionaries had to be initialized like:

Aggregations = new AggregationDictionary
{
    {"average_per_child", new AverageAggregation("average_per_child", ...)},
    {"max_per_child", new MaxAggregation("max_per_child", ...)},
    {"min_per_child", new MinAggregation("min_per_child", ...)},
}

and the && syntax simplified that a little bit:

Aggregations =
    new AverageAggregation("average_per_child", ...)
    && new MaxAggregation("max_per_child", ...)
    && new MinAggregation("min_per_child", ...)

In the latest version of the 8.x client, initialization is more straightforward anyways:

Aggregations = new Dictionary<string, Aggregation>
{
    {"average_per_child", Aggregation.Average(...)},
    {"max_per_child", new MaxAggregation(...)},
    {"min_per_child", new MinAggregation(...)},
}

from elasticsearch-net.

I hope you understand, that I can not bring this functionality back for technical reasons. I will make sure to document that in the release notes of 8.13.0.

from elasticsearch-net.

@flobernd, can you show me how to initialize Aggregation with aggregations in the latest 8.13.* client?

How "translate" this query in C#:

{
    "from": 0,
    "size": 100,
      "collapse": {
        "field": "uniqueId"
    },
    "query": {
        "match": {
            "Content": "test word"
        }
    },
    "aggregations": {
        "my-terms aggs": {
            "terms": {
                "field": "seqId",
                "size": 100
            },
          	"aggregations": {
        		"my-cardinality-agg": {
          			"cardinality": {
            			"field": "uniqueId"
          			}
       			}
      		}
        }
    }
}

• i must set and use "my-terms aggs" and "my-cardinality-agg" in code
• uniqueId and seqId are different fields
• search is performed on aliases that may contain duplicates

I tried

Aggregation.Terms(new TermsAggregation
{
  Field = Infer.Field("seqId"),
  Size = 100
}).Aggregations = new Dictionary<string, Aggregation>
{
  {
    "my-cardinality-agg",
    new CardinalityAggregation { Field = "uniqueId" }
  }
}

but how to set up name "my-terms aggs" in TermsAggregation in that case?

from elasticsearch-net.