Skip to content

Document special behaviour of ignore_malformed for geo_point mappings #125692

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

Merged

Conversation

craigtaverner
Copy link
Contributor

With geo_point fields, here is the special case of values that have a syntactically valid format, but the numerical values for latitude and longitude are out of range.

If ignore_malformed is false, an exception will be thrown as usual. But if it is true, the document will be indexed correctly, by normalizing the latitude and longitude values into the valid range. The special _ignored field will not be set. The original source document will remain as before, but indexed values, doc-values and stored fields will all be normalized.

Fixes #94807

This PR also adds a bunch of line-breaks to the geo-point.md file to make it easier to back-port this change to the 8.x branch. The line breaks have been chosen to mimic the original ASCIIDOC files, so the diff will be cleaner.

@craigtaverner craigtaverner added >docs General docs changes :Analytics/Geo Indexing, search aggregations of geo points and shapes Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) auto-backport Automatically create backport pull requests when merged v9.0.1 v9.1.0 labels Mar 26, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Docs Meta label for docs team label Mar 26, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-docs (Team:Docs)

@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

@colleenmcginnis colleenmcginnis requested review from leemthompo and removed request for colleenmcginnis March 26, 2025 16:55
@craigtaverner craigtaverner merged commit 1f6518f into elastic:main Apr 7, 2025
6 checks passed
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
9.0

craigtaverner added a commit to craigtaverner/elasticsearch that referenced this pull request Apr 7, 2025
…elastic#125692)

With `geo_point` fields, here is the special case of values that have a syntactically valid format, but the numerical values for `latitude` and `longitude` are out of range.

If `ignore_malformed` is `false`, an exception will be thrown as usual. But if it is `true`, the document will be indexed correctly, by normalizing the latitude and longitude values into the valid range. The special `_ignored` field will not be set. The original source document will remain as before, but indexed values, doc-values and stored fields will all be normalized.
elasticsearchmachine pushed a commit that referenced this pull request Apr 7, 2025
…#125692) (#126384)

With `geo_point` fields, here is the special case of values that have a syntactically valid format, but the numerical values for `latitude` and `longitude` are out of range.

If `ignore_malformed` is `false`, an exception will be thrown as usual. But if it is `true`, the document will be indexed correctly, by normalizing the latitude and longitude values into the valid range. The special `_ignored` field will not be set. The original source document will remain as before, but indexed values, doc-values and stored fields will all be normalized.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/Geo Indexing, search aggregations of geo points and shapes auto-backport Automatically create backport pull requests when merged >docs General docs changes Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) Team:Docs Meta label for docs team v9.0.1 v9.1.0
Projects
None yet
3 participants