Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Aggregations: Add `filters` aggregation #6118

Closed
uboness opened this Issue May 11, 2014 · 4 comments

Comments

Projects
None yet
5 participants
@uboness
Copy link
Contributor

commented May 11, 2014

Add a filters (note the s) aggregation that takes a list of filters and uses them to construct multiple buckets.

Technically the user could create multiple filter aggregations, however this gets messy if they have several sub aggregations as they would need to copy the contents into each of the top level aggregations.

@uboness uboness self-assigned this May 11, 2014

uboness added a commit to uboness/elasticsearch that referenced this issue May 11, 2014

Added Filters aggregation
A multi-bucket aggregation where multiple filters can be defined (each filter defines a bucket). The buckets will collect all the documents that match their associated filter.

This aggregation can be very useful when one wants to compare analytics between different criterias. It can also be accomplished using multiple definitions of the single filter aggregation, but here, the user will only need to define the sub-aggregations only once.

Closes elastic#6118

@jpountz jpountz added v1.3.0 labels Jun 13, 2014

@s1monw

This comment has been minimized.

Copy link
Contributor

commented Jul 9, 2014

@uboness I pushed this out to 1.4

@s1monw s1monw added v1.4.0 and removed v1.3.0 labels Jul 9, 2014

@uboness

This comment has been minimized.

Copy link
Contributor Author

commented Jul 9, 2014

+1

@rashidkpc

This comment has been minimized.

Copy link
Member

commented Jul 17, 2014

Out of curiosity, what is blocking this? Just review?

@uboness

This comment has been minimized.

Copy link
Contributor Author

commented Jul 17, 2014

@rashidkpc already talked with @colings86 about taking over and getting it in

@clintongormley clintongormley assigned colings86 and unassigned uboness Jul 18, 2014

colings86 added a commit to colings86/elasticsearch that referenced this issue Aug 1, 2014

Aggregations Added Filters aggregation
A multi-bucket aggregation where multiple filters can be defined (each filter defines a bucket). The buckets will collect all the documents that match their associated filter.

This aggregation can be very useful when one wants to compare analytics between different criterias. It can also be accomplished using multiple definitions of the single filter aggregation, but here, the user will only need to define the sub-aggregations only once.

Closes elastic#6118

uboness added a commit that referenced this issue Aug 1, 2014

Aggregations Added Filters aggregation
A multi-bucket aggregation where multiple filters can be defined (each filter defines a bucket). The buckets will collect all the documents that match their associated filter.

This aggregation can be very useful when one wants to compare analytics between different criterias. It can also be accomplished using multiple definitions of the single filter aggregation, but here, the user will only need to define the sub-aggregations only once.

Closes #6118

@spinscale spinscale changed the title Add `filters` aggregation Aggregations: Add `filters` aggregation Aug 4, 2014

uboness added a commit that referenced this issue Sep 8, 2014

Aggregations Added Filters aggregation
A multi-bucket aggregation where multiple filters can be defined (each filter defines a bucket). The buckets will collect all the documents that match their associated filter.

This aggregation can be very useful when one wants to compare analytics between different criterias. It can also be accomplished using multiple definitions of the single filter aggregation, but here, the user will only need to define the sub-aggregations only once.

Closes #6118
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.