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.

Sign up for GitHub

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

Jump to bottom

Aggregations: Geo bounds aggregation throwing ArrayIndexOutOfBoundsException on array resize #7556

Closed
owainb opened this issue Sep 3, 2014 · 2 comments · Fixed by #7565
Closed

Aggregations: Geo bounds aggregation throwing ArrayIndexOutOfBoundsException on array resize #7556

owainb opened this issue Sep 3, 2014 · 2 comments · Fixed by #7565
Assignees
@colings86
Labels
>bug v1.3.3 v1.4.0.Beta1 v2.0.0-beta1

Comments

@owainb
Copy link

owainb commented Sep 3, 2014

There is a bug in the collect method of org.elasticsearch.search.aggregations.metrics.geobounds.GeoBoundsAggregator. When it resizes all the arrays at the top of the method it resizes posLefts twice instead of resizing posRights. This causes and ArrayIndexOutOfBoundsException.
@owainb
Copy link
Author

owainb commented Sep 3, 2014

Should have mentioned that this is discussed in https://groups.google.com/forum/#!topic/elasticsearch/iUMK5GJ3JsQ

@colings86 colings86 self-assigned this Sep 3, 2014
@colings86 colings86 mentioned this issue Sep 3, 2014
Merged
colings86 added a commit that referenced this issue Sep 3, 2014
@colings86
Aggregations: Fixes resize bug in Geo bounds Aggregator
6245552 
Closes #7556
colings86 added a commit that referenced this issue Sep 3, 2014
@colings86
Aggregations: Fixes resize bug in Geo bounds Aggregator
881fd33 
Closes #7556
@colings86
Copy link
Contributor

@owainb thanks for reporting the bug

@colings86 colings86 changed the title Nested geo bounds aggregation throwing ArrayIndexOutOfBoundsException Aggregations: Geo bounds aggregation throwing ArrayIndexOutOfBoundsException on array resize Sep 3, 2014
colings86 added a commit that referenced this issue Sep 8, 2014
@colings86 @areek
Aggregations: Fixes resize bug in Geo bounds Aggregator
3c0794c 
Closes #7556
mute pushed a commit to mute/elasticsearch that referenced this issue Jul 29, 2015
@colings86
Aggregations: Fixes resize bug in Geo bounds Aggregator
c3e715b 
Closes elastic#7556
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@colings86 colings86

Labels
>bug v1.3.3 v1.4.0.Beta1 v2.0.0-beta1
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixes resize bug in Geo bounds Aggregator
2 participants
@colings86 @owainb