Skip to content

Conversation

@nreese
Copy link
Contributor

@nreese nreese commented Oct 4, 2019

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
Contributor

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

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

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

💔 Build Failed

Copy link
Contributor

@thomasneirynck thomasneirynck left a comment

Choose a reason for hiding this comment

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

@elasticmachine
Copy link
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
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