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

Deprecating request parameters of _analyze API in 5.x #20686

Merged

Conversation

johtani
Copy link
Contributor

@johtani johtani commented Sep 28, 2016

Deprecate request parameters of _analyze API in 5.x

Add deprecation log
Add deprecation description in the document
Add deprecation test in rest-api-test and unit test

Related #20246

Add deprecation log
Add deprecation description in  the document
Add deprecation test in rest-api-test and unit test

Closes elastic#20246
Copy link
Contributor

@s1monw s1monw left a comment

Choose a reason for hiding this comment

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

LGTM this change looks great! I also learned that we have a warning section in the rest test :)

Copy link
Member

@nik9000 nik9000 left a comment

Choose a reason for hiding this comment

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

LGTM too. The warnings section also applies to docs so the docs build will fail if it uses deprecated syntax and doesn't have a warning marker in the docs.

@johtani
Copy link
Contributor Author

johtani commented Sep 29, 2016

Thanks folks. Docs test is really cool! It is easy to find breaking changes in the docs.

@johtani johtani merged commit 6509714 into elastic:5.x Sep 29, 2016
karmi added a commit that referenced this pull request Oct 15, 2016
…dices Analyze" tests

As part of b6537d7, a check for the `Warnings` headers
has been added. Since the feature might be temporarily or permanently not supported by all clients
(as is the case with eg. the `headers` feature), we should allow them to skip the tests.

Related: #20246, #20686

CC @johtani
karmi added a commit that referenced this pull request Oct 18, 2016
…dices Analyze" tests

As part of b6537d7, a check for the `Warnings` headers
has been added. Since the feature might be temporarily or permanently not supported by all clients
(as is the case with eg. the `headers` feature), we should allow them to skip the tests.

Related: #20246, #20686

Closes #20953
karmi added a commit that referenced this pull request Oct 18, 2016
…dices Analyze" tests

As part of b6537d7, a check for the `Warnings` headers
has been added. Since the feature might be temporarily or permanently not supported by all clients
(as is the case with eg. the `headers` feature), we should allow them to skip the tests.

Related: #20246, #20686

Closes #20953
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>deprecation :Search/Analysis How text is split into tokens v5.1.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants