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

Composite Aggregation using Java Rest High-level Client #38020

Closed
mahendra-s opened this issue Jan 30, 2019 · 6 comments · Fixed by #38207
Closed

Composite Aggregation using Java Rest High-level Client #38020

mahendra-s opened this issue Jan 30, 2019 · 6 comments · Fixed by #38207

Comments

@mahendra-s
Copy link

Describe the feature: Composite Aggregation using Java Rest High-level Client
Elasticsearch version (bin/elasticsearch --version): 6.5.4
Our solution is built using Java Rest High-level client and we are using most of all services from this library.

As of now from "org.elasticsearch.client" % "elasticsearch-rest-high-level-client" % "6.5.4" there is no way to create composite aggregation request.

Please suggest alternative solution or any plan to incorporate this function

@mahendra-s
Copy link
Author

pinging @javanna

@javanna
Copy link
Member

javanna commented Jan 30, 2019

hi @mahendra-s have you tried using CompositeAggregationBuilder? Not sure if I am missing something but what made you think that composite aggs are not supported in the high-level REST client?

@tKe
Copy link

tKe commented Jan 30, 2019

Currently the aggregations are mostly discoverable through the org.elasticsearch.search.aggregations.AggregationBuilders helpers, however there isn't one for CompositeAggregationBuilder - this may be why @mahendra-s hasn't found the builder?

@javanna
Copy link
Member

javanna commented Jan 30, 2019

yes that could be it, I see that composite cannot be created through AggregationBuilders, thanks for pointing that out @tKe . All the aggregation builders can also be created directly, so that is possible and should solve your issue @mahendra-s .

@colings86 colings86 added the :Analytics/Aggregations Aggregations label Jan 31, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-analytics-geo

@mahendra-s
Copy link
Author

Correct. Thank you @javanna @tKe.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants