elasticsearch-query-builder
elasticsearch-query-builder copied to clipboard
erichard
Feature request: Composite Aggregation
Hello there,
I believe that "composite aggregations" are currently missing in v3 https://www.elastic.co/guide/en/elasticsearch/reference/current/search-aggregations-bucket-composite-aggregation.html
This is currently the class I built to handle that, it is very basic at the moment so all cases might not be covered. And I'm not sure if it follows the logic you have in your other aggregations but at least it might give an idea
<?php
declare(strict_types=1);
namespace Core\Search\Model\Search\DQL;
use Erichard\ElasticQueryBuilder\Aggregation\AbstractAggregation;
class CompositeAggregation extends AbstractAggregation
{
public function __construct(
string $name,
private array $aggregations,
private int $size = 10,
) {
parent::__construct($name, []);
}
protected function buildAggregation(): array
{
$build = [];
foreach ($this->aggregations as $aggregation) {
$build[] = [
$aggregation => [
'terms' => [
'field' => $aggregation,
],
],
];
}
return [
'sources' => $build,
'size' => $this->size,
];
}
protected function getType(): string
{
return 'composite';
}
}
Hi !
The logic in v3 is:
- Query and Aggregation MUST HAVE a constructor providing all required properties to build a valid object
- getter/setter can be added for others properties
For your Composite aggregation the only required property is source which seems similar but yet different from a real aggregation. Size should be handle with getter/setter.
Although similar, the terms value source doesn’t support the same set of parameters as the terms aggregation.
