Skip to content

Remove search exists api#13911

Closed
javanna wants to merge 1 commit intoelastic:masterfrom
javanna:enhancement/remove_search_exists
Closed

Remove search exists api#13911
javanna wants to merge 1 commit intoelastic:masterfrom
javanna:enhancement/remove_search_exists

Conversation

@javanna
Copy link
Copy Markdown
Contributor

@javanna javanna commented Oct 2, 2015

With #13910 we deprecate _search/exists, we can then remove the api from the master branch.

Closes #13682

@javanna javanna added >breaking review :Search/Search Search-related issues that do not fall into other categories v5.0.0-alpha1 labels Oct 2, 2015
@jpountz
Copy link
Copy Markdown
Contributor

jpountz commented Oct 3, 2015

LGTM

@jpountz
Copy link
Copy Markdown
Contributor

jpountz commented Oct 3, 2015

Should we add a note to the migration guide?

@javanna javanna force-pushed the enhancement/remove_search_exists branch from d71acfc to 3ec445d Compare October 19, 2015 08:35
@javanna
Copy link
Copy Markdown
Contributor Author

javanna commented Oct 19, 2015

I added the missing note to the migration guide and rebased, should be ready

@jpountz
Copy link
Copy Markdown
Contributor

jpountz commented Oct 21, 2015

LGTM again

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

Labels

>breaking :Search/Search Search-related issues that do not fall into other categories v5.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants