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

Remove deprecated date_histo intervals #66715

Closed
polyfractal opened this issue Dec 21, 2020 · 2 comments
Closed

Remove deprecated date_histo intervals #66715

polyfractal opened this issue Dec 21, 2020 · 2 comments
Assignees
Labels
:Analytics/Aggregations Aggregations >breaking Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo)

Comments

@polyfractal
Copy link
Contributor

We split date_histo intervals between calendar and fixed intervals in #33727. At the same time, we deprecated the older, unified interval parameter.

It's been deprecated since 7.2, so we should be good to remove from 8.0 in my opinion. There is a non-negligible amount of code to handle the old intervals and it'd be great to clean that up.

This issue is a placeholder so we don't forget to remove it as 8.0 grows nearer :)

@elasticmachine elasticmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Dec 21, 2020
@elasticmachine
Copy link
Collaborator

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

@not-napoleon not-napoleon self-assigned this Jun 9, 2021
@not-napoleon
Copy link
Member

I resolved this with #75000, I just forgot to tell github to close this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/Aggregations Aggregations >breaking Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo)
Projects
None yet
Development

No branches or pull requests

3 participants