Skip to content

[7.11] Revert "Remove aggregation's postCollect phase (#68942)#69066

Merged
imotov merged 2 commits intoelastic:7.11from
imotov:issue-66876-sorting-on-nested-7.11
Feb 18, 2021
Merged

[7.11] Revert "Remove aggregation's postCollect phase (#68942)#69066
imotov merged 2 commits intoelastic:7.11from
imotov:issue-66876-sorting-on-nested-7.11

Conversation

@imotov
Copy link
Copy Markdown
Contributor

@imotov imotov commented Feb 16, 2021

This partially reverts #64016 and and adds #67839 and adds
additional tests that would have caught issues with the changes
in #64016. It's mostly Nik's code, I am just cleaning things up
a bit.

Co-authored-by: Nik Everett nik9000@gmail.com

This partially reverts elastic#64016 and and adds elastic#67839 and adds
additional tests that would have caught issues with the changes
in elastic#64016. It's mostly Nik's code, I am just cleaning things up
a bit.

Co-authored-by: Nik Everett nik9000@gmail.com
@elasticmachine elasticmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Feb 16, 2021
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-analytics-geo (Team:Analytics)

@imotov imotov changed the title [7.x] Revert "Remove aggregation's postCollect phase (#68942) [7.11] Revert "Remove aggregation's postCollect phase (#68942) Feb 16, 2021
@imotov
Copy link
Copy Markdown
Contributor Author

imotov commented Feb 18, 2021

@elasticmachine update branch

@imotov imotov merged commit 27e59ab into elastic:7.11 Feb 18, 2021
@imotov imotov deleted the issue-66876-sorting-on-nested-7.11 branch March 27, 2021 00:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/Aggregations Aggregations backport Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants