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

The `nested` aggregator should also resolve and use the parentFilter of the closest `reverse_nested` aggregator. #7048

Conversation

Projects
None yet
3 participants
@martijnvg
Copy link
Member

commented Jul 27, 2014

PR for #6994

@jpountz

This comment has been minimized.

Copy link
Contributor

commented Jul 28, 2014

LGTM

@jpountz jpountz removed the review label Jul 28, 2014

The `nested` aggregator should also resolve and use the parentFilter …
…of the closest `reverse_nested` aggregator.

Closes #6994
Closes #7048

martijnvg added a commit that referenced this pull request Jul 28, 2014

The `nested` aggregator should also resolve and use the parentFilter …
…of the closest `reverse_nested` aggregator.

Closes #6994
Closes #7048

martijnvg added a commit that referenced this pull request Jul 28, 2014

The `nested` aggregator should also resolve and use the parentFilter …
…of the closest `reverse_nested` aggregator.

Closes #6994
Closes #7048

@martijnvg martijnvg merged commit 2e9ee5c into elastic:master Jul 28, 2014

@clintongormley clintongormley changed the title The `nested` aggregator should also resolve and use the parentFilter of the closest `reverse_nested` aggregator. Aggregations: The `nested` aggregator should also resolve and use the parentFilter of the closest `reverse_nested` aggregator. Jul 28, 2014

@martijnvg martijnvg deleted the martijnvg:bugs/reverse_nested_also_detected_closest_reverse_nested_agg branch May 18, 2015

@clintongormley clintongormley changed the title Aggregations: The `nested` aggregator should also resolve and use the parentFilter of the closest `reverse_nested` aggregator. The `nested` aggregator should also resolve and use the parentFilter of the closest `reverse_nested` aggregator. Jun 7, 2015

mute pushed a commit to mute/elasticsearch that referenced this pull request Jul 29, 2015

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.