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

PercentilesAggregation is missing in the 8.x client #7879

Closed
gise88 opened this issue Aug 8, 2023 · 4 comments · Fixed by #7953
Closed

PercentilesAggregation is missing in the 8.x client #7879

gise88 opened this issue Aug 8, 2023 · 4 comments · Fixed by #7953
Labels
8.x Relates to 8.x client version

Comments

@gise88
Copy link

gise88 commented Aug 8, 2023

Is your feature request related to a problem? Please describe.
Percentiles metrics aggregation is missing in new client v8.*
https://www.elastic.co/guide/en/elasticsearch/reference/current/search-aggregations-metrics-percentile-aggregation.html

@gise88 gise88 added the Feature label Aug 8, 2023
@flobernd flobernd added the 8.x Relates to 8.x client version label Aug 8, 2023
@gise88
Copy link
Author

gise88 commented Sep 8, 2023

@flobernd Any news about this? Could you please provide an ETA? We're migrating old code, and this particular issue is causing significant troubles! 😕

@flobernd
Copy link
Member

flobernd commented Sep 8, 2023

This aggregation got reintroduced in #7923. Will be included in the next release.

@gise88
Copy link
Author

gise88 commented Sep 28, 2023

@flobernd Hey once again, we're having some issues with this PercentilesAggregation.. We used like in the screenshot but the SearchAsync throwes that tdigest_percentiles is not supported. Should we write the query/aggregation in some other ways?

image

@flobernd
Copy link
Member

Hi @gise88!

I'm sorry, he accessor method was not generated. I already fixed this locally and planning to include it in the next release (which will most likely happen tomorrow).

Not all tests are currently ported from NEST to the new client which caused me to miss this 😕

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8.x Relates to 8.x client version
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants