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

[4.x] Deprecate ascending _count sort from terms agg #6833

Closed
rashidkpc opened this issue Apr 8, 2016 · 5 comments
Closed

[4.x] Deprecate ascending _count sort from terms agg #6833

rashidkpc opened this issue Apr 8, 2016 · 5 comments
Assignees

Comments

@rashidkpc
Copy link
Contributor

Its being removed from Elasticsearch so we need to start issuing a deprecation notice once the final decision is made: elastic/elasticsearch#17614

@epixa
Copy link
Contributor

epixa commented Apr 8, 2016

I'm preempting this with a blocker label for 5.0.0 just so it doesn't slip between the cracks. We can ditch the labels if the corresponding change to ES doesn't move forward.

@jimczi
Copy link

jimczi commented Jun 17, 2016

@epixa @rashidkpc:
The support for sorting terms aggregation by ascending count has been removed in ES master:
elastic/elasticsearch#18940

@epixa
Copy link
Contributor

epixa commented Jun 17, 2016

Ah crap, I figured this was being deprecated in ES first.

@epixa epixa added v4.6.0 and removed v5.0.0 labels Jun 17, 2016
@epixa epixa changed the title Deprecate ascending _count sort from terms agg [4.x] Deprecate ascending _count sort from terms agg Jun 17, 2016
@epixa
Copy link
Contributor

epixa commented Jun 17, 2016

I updated this ticket to add deprecation warnings to 4.x, and I created #7494 to remove the capabilities entirely from master.

@ycombinator
Copy link
Contributor

Resolved in #8050.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants