Skip to content

[Backport 1.x] Support first and last parameter for missing bucket ordering in composite aggregation#2049

Merged
saratvemulapalli merged 1 commit intoopensearch-project:1.xfrom
penghuo:1.x-bp-1899
Feb 15, 2022
Merged

[Backport 1.x] Support first and last parameter for missing bucket ordering in composite aggregation#2049
saratvemulapalli merged 1 commit intoopensearch-project:1.xfrom
penghuo:1.x-bp-1899

Conversation

@penghuo
Copy link
Copy Markdown
Contributor

@penghuo penghuo commented Feb 3, 2022

Description

Backporting #1942 and #2005 to 1.x branch.

Passed, ./gradlew ':server:test' -Dtests.iters=100 --tests "org.opensearch.search.aggregations.bucket.composite.CompositeAggregatorTests"
Passed, ./gradlew precommit

Issues Resolved

#1899

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • [x ] New functionality has javadoc added
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

…site aggregation (opensearch-project#1942)

Support for "first" and "last" parameters for missing bucket ordering in composite aggregation.
By default, if order is asc, missing_bucket at first, if order is desc, missing_bucket at last. If
missing_order is "first" or "last", regardless order, missing_bucket is at first or last respectively.

Signed-off-by: Peng Huo <penghuo@gmail.com>
@opensearch-ci-bot
Copy link
Copy Markdown
Collaborator

Can one of the admins verify this patch?

@opensearch-ci-bot
Copy link
Copy Markdown
Collaborator

✅   Gradle Check success 40320b8
Log 2218

Reports 2218

@penghuo penghuo changed the title Support first and last parameter for missing bucket ordering in composite aggregation [Backport 1.x] Support first and last parameter for missing bucket ordering in composite aggregation Feb 15, 2022
@saratvemulapalli saratvemulapalli added backport PRs or issues specific to backporting features or enhancments v1.3.0 enhancement Enhancement or improvement to existing feature or request labels Feb 15, 2022
@saratvemulapalli saratvemulapalli merged commit 5b27136 into opensearch-project:1.x Feb 15, 2022
andrross added a commit to andrross/OpenSearch that referenced this pull request Feb 16, 2022
…in composite aggregation (opensearch-project#1942) (opensearch-project#2049)"

This reverts commit 5b27136.

Signed-off-by: Andrew Ross <andrross@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport PRs or issues specific to backporting features or enhancments enhancement Enhancement or improvement to existing feature or request v1.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants