Skip to content

[Maps] retrieve geo_point value from docvalue_fields instead of _source - #47389

Merged
nreese merged 5 commits into
elastic:masterfrom
nreese:geo_field_docvalue_field
Oct 11, 2019
Merged

nreese merged 5 commits into
elastic:masterfrom
nreese:geo_field_docvalue_field

Conversation

@nreese

@nreese nreese commented Oct 4, 2019

Copy link
Copy Markdown
Contributor

fixes #45270

This PR updates the Elasticsearch search request to request geo_point geometry from docvalue_fields instead of _source. This way, docvalue_fields will come back in a lat,lon string format regardless of how the field is stored in _source.

Run the following in console to load a test data set with WKT to test the PR out.

PUT test
{}

PUT test/_mapping
{
  "properties": {
    "location": {
      "type": "geo_point"
    }
  }
}

PUT test/_doc/1
{
    "location": "POINT (-71.34 41.12)"
}

PUT test/_doc/2
{
    "location": ["POINT (-68 41.12)", "POINT (-66 41.12)"]
}

@nreese nreese added release_note:enhancement Team:Geo Former Team Label for Geo Team. Now use Team:Presentation v8.0.0 v7.5.0 labels Oct 4, 2019
@elasticmachine

Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-gis (Team:Geo)

@elasticmachine

Copy link
Copy Markdown
Contributor

💔 Build Failed

@elasticmachine

Copy link
Copy Markdown
Contributor

💔 Build Failed

@nreese
nreese requested a review from thomasneirynck October 10, 2019 17:11
@elasticmachine

Copy link
Copy Markdown
Contributor

💔 Build Failed

@thomasneirynck thomasneirynck left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@elasticmachine

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@nreese
nreese merged commit 4fc7551 into elastic:master Oct 11, 2019
nreese added a commit to nreese/kibana that referenced this pull request Oct 11, 2019
…ce (elastic#47389)

* [Maps] retrieve geo_point value from docvalue_fields instead of _source

* add functional test ensuring _search request only pulls what is needed

* clean up i18n translation fiels
nreese added a commit that referenced this pull request Oct 11, 2019
…ce (#47389) (#47917)

* [Maps] retrieve geo_point value from docvalue_fields instead of _source

* add functional test ensuring _search request only pulls what is needed

* clean up i18n translation fiels
patrykkopycinski pushed a commit to patrykkopycinski/kibana that referenced this pull request May 6, 2026
…ce (elastic#47389)

* [Maps] retrieve geo_point value from docvalue_fields instead of _source

* add functional test ensuring _search request only pulls what is needed

* clean up i18n translation fiels
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:enhancement Team:Geo Former Team Label for Geo Team. Now use Team:Presentation v7.5.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Maps] support WKT for geo_point fields

3 participants