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

Fix GeoPoint FieldStats ternary logic bug #25211

Merged
merged 1 commit into from Jun 14, 2017

Conversation

Projects
None yet
3 participants
@nknize
Member

nknize commented Jun 14, 2017

The ternary logic in prefixCodedToGeoPoint was reversed such that numericEncoded GeoPoints were using the decoding logic for GeoCoded points and vice versa. This PR fixes that boneheaded bug.

closes #24275

@jimczi

jimczi approved these changes Jun 14, 2017

Fix GeoPoint FieldStats ternary logic bug
The ternary logic in prefixCodedToGeoPoint was reversed such that numericEncoded GeoPoints were using the decoding logic for GeoCoded points and vice versa. This commit fixes that boneheaded bug.

@nknize nknize merged commit e763b43 into elastic:5.x Jun 14, 2017

0 of 2 checks passed

CLA Error when checking the CLA status
Details
elasticsearch-ci Build started sha1 is merged.
Details

@clintongormley clintongormley added v5.5.0 and removed v5.5.1 labels Jun 29, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment