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

merge GeoPoint specific mapping properties #5506

Closed
wants to merge 1 commit into from

Conversation

kzwang
Copy link
Contributor

@kzwang kzwang commented Mar 24, 2014

closes #5505
I assume only validate_lat and validate_lon can be updated, other fields will throw exceptions. Please let me know if I was wrong.

Also added precision_step to the document page

@jpountz jpountz self-assigned this Mar 24, 2014
@jpountz
Copy link
Contributor

jpountz commented Mar 24, 2014

This change is good, I just merged it via bfd3236.

Thanks!

@jpountz jpountz closed this Mar 24, 2014
@kzwang kzwang deleted the feature/geopoint_mapper branch March 24, 2014 09:51
@clintongormley clintongormley added >bug v2.0.0-beta1 v1.2.0 :Search/Mapping Index mappings, including merging and defining field types labels Jun 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug :Search/Mapping Index mappings, including merging and defining field types v1.2.0 v2.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GeoPointFieldMapper doesn't merge GeoPoint specific properties
3 participants