-
Notifications
You must be signed in to change notification settings - Fork 803
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
Add support for the multi_terms aggregation #1512
Comments
This aggregation is new in Elasticsearch 7.12 and hasn't been implemented in Elasticsearch-DSL natively yet. |
No need to close this until it's implemented. |
same problem with
While waiting for it to be implemented, is there a workaround to tell DSL "don't bother if you don't understand it, just add the parameter in the elastic search request" |
Aims to tackle second issue described here: elastic#1512
Any idea on timeline for the 'multi_terms' aggregation being natively supported in the elasticsearch dsl? Cheers! |
I've just sent a pr for this issue, could you review it please? @sethmlarson Also, should i create a new issue to implement "combined_fields" query? |
Is there a workaround in the meantime? |
I think there isn't. You can write it natively. @EvieGillie |
could you please consider adding support to the "combined_fields" query? @Telomeraz |
@Telomeraz should I create a new issue for combined_fields, as this one is closed? |
@vipulg13 Yes, you can create a new one. |
Good day!
How can i use multu-terms aggregation ?
I have got :
elasticsearch_dsl.exceptions.UnknownDslObject: DSL class
multi_termsdoes not exist in agg.
Is there a method implemented?
The text was updated successfully, but these errors were encountered: