Skip to content

[7.x] Support "_first" and "_last" ordering of missing values in composite aggs (#76740)#77620

Merged
elasticsearchmachine merged 2 commits intoelastic:7.xfrom
Luegg:backport/7.x/pr-76740
Sep 13, 2021
Merged

[7.x] Support "_first" and "_last" ordering of missing values in composite aggs (#76740)#77620
elasticsearchmachine merged 2 commits intoelastic:7.xfrom
Luegg:backport/7.x/pr-76740

Conversation

@Luegg
Copy link
Copy Markdown
Contributor

@Luegg Luegg commented Sep 13, 2021

Backports the following commits to 7.x:

…aggs (elastic#76740)

* Support "first" and "last" ordering of missing values in composite aggs

* skip API compat tests for 7.16

* use proper versions for bwc

* use "missing_order" parameter instead of "missing"

* address review comments
# Conflicts:
#	server/src/main/java/org/elasticsearch/search/aggregations/bucket/composite/CompositeValuesSourceBuilder.java
@Luegg Luegg added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Sep 13, 2021
@elasticsearchmachine elasticsearchmachine merged commit 5087b9f into elastic:7.x Sep 13, 2021
@Luegg Luegg deleted the backport/7.x/pr-76740 branch September 14, 2021 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport v7.16.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants