elastic / elasticsearch

Free and Open, Distributed, RESTful Search Engine
https://www.elastic.co/products/elasticsearch
Other
69.45k stars 24.58k forks source link

Add min_doc_count option to geo grid aggregations #50481

Closed yutaka-nakano closed 1 year ago

yutaka-nakano commented 4 years ago

Would it be possible to add min_doc_count option to geohash grid aggregation and geotile grid aggrigation?
We use Elasticsearch and Kibana for geospatial data analysis. min_doc_count option would be very helpful for us to mask privacy information on dashboards with maps. It would also be helpful for excluding outliers.

We created a Pull Request for this feature. #50480 Could you check it?

Update (January 2023) min_doc_count should also be added to

elasticmachine commented 4 years ago

Pinging @elastic/es-analytics-geo (:Analytics/Geo)

gilesagnel commented 2 years ago

Hi @wchaparro Can i pick this up?

elasticsearchmachine commented 1 year ago

Pinging @elastic/es-analytics-geo (Team:Analytics)

nreese commented 1 year ago

There have been requests in Kibana for filtering geo aggregation buckets

craigtaverner commented 1 year ago

This seems easy enough, as long as we exclude support for min_doc_count=0.

timothy-choi commented 1 year ago

Hi @yutaka-nakano, is this still open? I would like to fix this issue.

iverase commented 1 year ago

@timDINOS yes it is still open and I think nobody is actively working on that so please feel free to work on it.

Kiriakos1998 commented 1 year ago

Hello, @timothy-choi are you working on this issue? I am looking for my first issue to attempt my first contribution and If no one is working with this one I would love to give it a shot.

iverase commented 1 year ago

We discussed this issue internally and we decided we don't want to pursue this idea anymore. Similarly to https://github.com/elastic/elasticsearch/issues/54649, we think Bucket selectors are the correct solution to do this type of operation and therefore I am closing this issue.