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

[Maps] disable fit to data for geo_shape fields until elasticsearch supports a way to get the bounds #31638

Merged
merged 4 commits into from
Feb 21, 2019

Conversation

nreese
Copy link
Contributor

@nreese nreese commented Feb 20, 2019

fixes #31589

get_bounds aggregation only support geo_point. There is no way to get the bounds for a geo_shape so this PR just disables the option. The PR also properly handles any geo_bounds errors.

screen shot 2019-02-20 at 3 41 04 pm

@nreese nreese added v7.0.0 [Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation v8.0.0 v6.7.0 v7.2.0 labels Feb 20, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-gis

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

it('should register elasticsearch request in inspector', async () => {
const hits = await getHits();
expect(hits).to.equal('6');
it('should apply query to fit to bounds', async () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

thanks for introducing this

try {
const esResp = await searchSource.fetch();
esBounds = _.get(esResp, 'aggregations.1.bounds');
} catch(error) {
Copy link
Contributor

Choose a reason for hiding this comment

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

👍 fallback

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@nreese nreese merged commit b4ece8f into elastic:master Feb 21, 2019
nreese added a commit to nreese/kibana that referenced this pull request Feb 21, 2019
…upports a way to get the bounds (elastic#31638)

* [Maps] disable fit to data for geo_shape fields until elasticsearch supports a way to get the bounds

* add function test for fit to bounds

* add jest test for layer_toc_actions component

* review feedback
nreese added a commit to nreese/kibana that referenced this pull request Feb 21, 2019
…upports a way to get the bounds (elastic#31638)

* [Maps] disable fit to data for geo_shape fields until elasticsearch supports a way to get the bounds

* add function test for fit to bounds

* add jest test for layer_toc_actions component

* review feedback
nreese added a commit that referenced this pull request Feb 21, 2019
…upports a way to get the bounds (#31638) (#31700)

* [Maps] disable fit to data for geo_shape fields until elasticsearch supports a way to get the bounds

* add function test for fit to bounds

* add jest test for layer_toc_actions component

* review feedback
nreese added a commit that referenced this pull request Feb 21, 2019
…upports a way to get the bounds (#31638) (#31699)

* [Maps] disable fit to data for geo_shape fields until elasticsearch supports a way to get the bounds

* add function test for fit to bounds

* add jest test for layer_toc_actions component

* review feedback
@bhavyarm
Copy link
Contributor

@nreese when I add this index pattern - it doesn't show as one of the documents in elasticsearch documents layer in maps. Is that expected?

POST gps_test/_doc/1
{
"poslist":
{ "type": "LINESTRING",
"coordinates": [[10.491832, 59.941252],[10.491833, 59.941252],[10.491898, 59.941263]]
}
}

@nreese
Copy link
Contributor Author

nreese commented Feb 28, 2019

What is the mapping for poslist?

@nreese nreese added non-issue Indicates to automation that a pull request should not appear in the release notes release_note:skip Skip the PR/issue when compiling release notes and removed non-issue Indicates to automation that a pull request should not appear in the release notes labels Mar 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation release_note:skip Skip the PR/issue when compiling release notes v6.7.0 v7.0.0 v7.2.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Maps] adding layers with geo_shape linestring bug
4 participants