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

no support for reverse_nested aggregations #59

Open
jameslamb opened this issue May 9, 2018 · 1 comment
Open

no support for reverse_nested aggregations #59

jameslamb opened this issue May 9, 2018 · 1 comment

Comments

@jameslamb
Copy link
Collaborator

jameslamb commented May 9, 2018

Per discussion in #58 ... es_search does not currently support reverse_nested aggregations. Well, technically it might support them but we have no tests around that.

To be honest I have no idea how those aggregations work, but would love for someone knowledgable to try adding support. Reference on them:

https://www.elastic.co/guide/en/elasticsearch/reference/current/search-aggregations-bucket-reverse-nested-aggregation.html

Requirements for closing this issue:

  1. Add code to our integration tests that create a structure in our test ES cluster that even allows this type of query (might require figuring out which versions of Elasticsearch actually support reverse_nested stuff)
  2. Create tests that confirm expected results for queries with reverse_nested stuff in them
@jameslamb
Copy link
Collaborator Author

A fix for this is not going to go out on the next release.

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

No branches or pull requests

1 participant