Skip to content

Conversation

@jtibshirani
Copy link
Contributor

We already started issuing a deprecation warning in #36802, so this PR just adds a test and tweaks the message to cover aggregations.

@jtibshirani jtibshirani added :Search Foundations/Mapping Index mappings, including merging and defining field types >deprecation v7.0.0 labels Jan 4, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search

Copy link
Contributor

@jimczi jimczi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jtibshirani jtibshirani merged commit 0bac64f into elastic:master Jan 4, 2019
@jtibshirani jtibshirani deleted the deprecate-types-in-aggregations branch January 4, 2019 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>deprecation :Search Foundations/Mapping Index mappings, including merging and defining field types v7.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants