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

Fix parsing RareTerms aggregation response in RestHighLevelClient #65144

Merged
merged 1 commit into from
Nov 17, 2020

Conversation

swallez
Copy link
Member

@swallez swallez commented Nov 17, 2020

Backport of #64454

  • Add LongRareTerms and StringRareTerms to the DefaultNamedXContents,
    ensure that the response of RareTerms aggregation can be parsed
    correctly.
  • Add testSearchWithRareTermsAgg method to test the response of
    RareTerms aggregation can be parsed correctly.
  • Add some test code to ensure the AggregationsTests can execute
    successfully.

…ent (elastic#65103)

Backport of elastic#64454

- Add LongRareTerms and StringRareTerms to the DefaultNamedXContents,
  ensure that the response of RareTerms aggregation can be parsed
  correctly.
- Add testSearchWithRareTermsAgg method to test the response of
  RareTerms aggregation can be parsed correctly.
- Add some test code to ensure the AggregationsTests can execute
  successfully.

Co-authored-by: bellengao <gbl_long@163.com>
@swallez swallez changed the title Fix parsing RareTerms aggregation response in RestHighLevelClient (#65103) Fix parsing RareTerms aggregation response in RestHighLevelClient Nov 17, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features (Team:Core/Features)

@swallez swallez merged commit b2475f9 into elastic:7.10 Nov 17, 2020
@swallez swallez deleted the backport-64454-7.10 branch November 17, 2020 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Team:Data Management Meta label for data/management team v7.10.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants