Skip to content

Conversation

@imotov
Copy link
Contributor

@imotov imotov commented Dec 20, 2018

Adds example on translating geohashes returned by geohashgrid agg
as bucket keys into geo bounding box filters in elasticsearch as well
as 3rd party applications.

Closes #36413

Adds example on translating geohashes returned by geohashgrid agg
as bucket keys into geo bounding box filters in elasticsearch as well
as 3rd party applications.

Closes elastic#36413
@imotov imotov added >docs General docs changes :Analytics/Geo Indexing, search aggregations of geo points and shapes v7.0.0 v6.6.0 v6.7.0 labels Dec 20, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-analytics-geo

@imotov imotov requested review from iverase and nknize December 20, 2018 18:00
Copy link
Contributor

@polyfractal polyfractal left a comment

Choose a reason for hiding this comment

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

Tiny comment which you can ignore if you like. LGTM!

I'm happy how small the JS snippet turned out to be :)

--------------------------------------------------
// TESTRESPONSE[s/\.\.\./"took": $body.took,"_shards": $body._shards,"hits":$body.hits,"timed_out":false,/]

For "zooming in" in the system that don't support geohashes, the bucket keys should be translated into bounding boxes using
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: perhaps For "zooming in" on systems that don't [...] ? The double "in" made me stumble reading it :)

@imotov imotov merged commit d6acd8e into elastic:master Jan 3, 2019
@imotov
Copy link
Contributor Author

imotov commented Jan 3, 2019

Because #30698 is breaking and was merged only into 7.0, the first example can be only merged into 7.x.

imotov added a commit that referenced this pull request Jan 3, 2019
Adds an example on translating geohashes returned by geohashgrid
agg as bucket keys into geo bounding box filters in elasticsearch as well
as 3rd party applications.

Closes #36413
imotov added a commit that referenced this pull request Jan 3, 2019
Adds an example on translating geohashes returned by geohashgrid
agg as bucket keys into geo bounding box filters in elasticsearch as well
as 3rd party applications.

Closes #36413
@jimczi jimczi added v7.0.0-beta1 and removed v7.0.0 labels Feb 7, 2019
@imotov imotov deleted the issue-36413-geohash-docs branch May 1, 2020 22:18
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 >docs General docs changes v6.6.0 v6.7.0 v7.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants