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.

Sign up for GitHub

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

Jump to bottom

Field stats API skips geo_point fields #20707

Closed
spalger opened this issue Sep 30, 2016 · 4 comments
Closed

Field stats API skips geo_point fields #20707

spalger opened this issue Sep 30, 2016 · 4 comments
Labels
>bug :Search Foundations/Mapping Index mappings, including merging and defining field types Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch

Comments

@spalger
Copy link
Contributor

spalger commented Sep 30, 2016

Elasticsearch version: master

Plugins installed: none

JVM version: 1.8.0_102

OS version: macOS Sierra

Description of the problem including expected versus actual behavior: fields of type geo_point are not described in the _field_stats output

Steps to reproduce:
sense requests: https://gist.github.com/spalger/72050d99ee9dc5a4ee8d1186e9cc05c9
which results in a matching filter, but no pin.location field listed in the field_stats output
@clintongormley clintongormley added >bug :Search Foundations/Mapping Index mappings, including merging and defining field types v5.0.0 labels Sep 30, 2016
@clintongormley
Copy link
Contributor

@jpountz could you take a look at this?

spalger pushed a commit to spalger/kibana that referenced this issue Sep 30, 2016
[indexPatterns] add workaround for elastic/elasticsearch#20707
22c6f7c
This was referenced Sep 30, 2016
Merged
Closed
elastic-jasper added a commit to elastic/kibana that referenced this issue Sep 30, 2016
@elastic-jasper
Backport PR #8500
7a13d9e 
---------

**Commit 1:**
Update expected table headers to fix functional test

* Original sha: f65c9b5
* Authored by Matthew Bargar <[email protected]> on 2016-09-26T12:40:41Z

**Commit 2:**
[indexPatterns] add workaround for elastic/elasticsearch#20707

* Original sha: 22c6f7c
* Authored by spalger <[email protected]> on 2016-09-30T10:21:07Z
This was referenced Sep 30, 2016
Merged
Merged
elastic-jasper added a commit to elastic/kibana that referenced this issue Sep 30, 2016
@elastic-jasper
Backport PR #8500
203353d 
---------

**Commit 1:**
Update expected table headers to fix functional test

* Original sha: f65c9b5
* Authored by Matthew Bargar <[email protected]> on 2016-09-26T12:40:41Z

**Commit 2:**
[indexPatterns] add workaround for elastic/elasticsearch#20707

* Original sha: 22c6f7c
* Authored by spalger <[email protected]> on 2016-09-30T10:21:07Z
@jpountz
Copy link
Contributor

jpountz commented Oct 3, 2016

The current behaviour of this API is to ignore all fields for which we do not know how to compute stats. Geo points are fixable, but should we add other fields like geo shapes as well as meta fields such as _index? Those are currently ignored as well.

@thomasneirynck thomasneirynck mentioned this issue Oct 3, 2016
Closed
@spalger
Copy link
Contributor Author

spalger commented Oct 3, 2016 •
edited
Loading

My naive opinion is that all values that work as "field" identifiers in queries or aggregations should be included in the stats.

@jpountz jpountz assigned nknize and unassigned jpountz Oct 24, 2016
@nknize nknize mentioned this issue Jan 12, 2017
Closed
airow pushed a commit to airow/kibana that referenced this issue Feb 16, 2017
@elastic-jasper
Backport PR elastic#8500
6d4ca82 
---------

**Commit 1:**
Update expected table headers to fix functional test

* Original sha: 8392fa0be6cb7bc2f23661ca921e15f07959fc85 [formerly f65c9b5]
* Authored by Matthew Bargar <[email protected]> on 2016-09-26T12:40:41Z

**Commit 2:**
[indexPatterns] add workaround for elastic/elasticsearch#20707

* Original sha: 351c30b1541e72065abf8f3dcc4c3cee9a97fc1f [formerly 22c6f7c]
* Authored by spalger <[email protected]> on 2016-09-30T10:21:07Z


Former-commit-id: 7a13d9e
@Bargs Bargs mentioned this issue Apr 24, 2017
Closed
@javanna
Copy link
Member

javanna commented Mar 16, 2018

This seems to have been fixed by #21947. After that though the field stats API has been removed.

@javanna javanna closed this as completed Mar 16, 2018
@javanna javanna added the Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch label Jul 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
>bug :Search Foundations/Mapping Index mappings, including merging and defining field types Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@clintongormley @jpountz @nknize @javanna @spalger