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

Fail queries that have two aggregations with the same name. #6258

Merged
merged 1 commit into from May 21, 2014

Conversation

@jpountz
Copy link
Contributor

commented May 21, 2014

Close #6255

@jpountz jpountz added v1.3.0 and removed v1.3.0 labels May 21, 2014
@colings86

This comment has been minimized.

Copy link
Member

commented May 21, 2014

Looks good to me

@s1monw

This comment has been minimized.

Copy link
Contributor

commented May 21, 2014

LGTM - I think this can go into 1.2 and 1.1.2 if applicable

@jpountz jpountz added v1.2.0 labels May 21, 2014
@jpountz jpountz merged commit 34f7bd1 into elastic:master May 21, 2014
@jpountz jpountz deleted the jpountz:fix/same_name_aggs branch May 21, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.