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

Jump to bottom

deprecate index status #4854

Closed
kevinkluge opened this issue Jan 22, 2014 · 5 comments
Closed

deprecate index status #4854

kevinkluge opened this issue Jan 22, 2014 · 5 comments
Assignees
@spinscale
Labels
v1.2.0

Comments

@kevinkluge
Copy link
Member

With the addition of the recovery API in issue #4637 the index status API call will be redundant. We could then deprecate and eventually eliminate the call.
@s1monw s1monw added v1.2.0 and removed v1.1.0 labels Mar 20, 2014
clintongormley added a commit that referenced this issue Apr 28, 2014
@clintongormley
Removed indices-status docs.
4b9f1d2 
Related #4854
@clintongormley
Copy link

I've deprecated the indices status API in the 1.x branch d12418e , and removed it in the master branch 4b9f1d2

The code also needs to be deprecated/removed.

clintongormley added a commit that referenced this issue Apr 28, 2014
@clintongormley
Removed spec and YAML tests for indices.status
2dfc77a 
Related #4854
@kevinkluge
Copy link
Member Author

Alex, can you remove this code from master (master only)?

Adding breaking label for master / 2.0. No breaking change for 1.x ; only deprecation.

@spinscale spinscale mentioned this issue May 6, 2014
Merged
@spinscale
Copy link
Contributor

Created the above PR for master only, which completely removes the index status API (also on java level). I think it makes more sense to create an additional PR for the 1.x branch, where everything related to this API is marked as deprecated.

@spinscale
Copy link
Contributor

Referenced a commit of the 1.x branch (cant create a PR from it, as it somehow goes against master instead of 1.x and shows tons of commits), which adds @Deprecated annotations to all of the index status classes, quick review is appreciated and then I will get in both. See spinscale@d6edde7

spinscale added a commit to spinscale/elasticsearch that referenced this issue May 7, 2014
@spinscale
Deprecate Index Status API
471a9a5 
Elasticsearch 1.1 introduced the recovery API, which replaces the
functionality of the index status API.
This marks the index status API as deprecated in order to remove it in the future.

Relates elastic#4854
spinscale added a commit to spinscale/elasticsearch that referenced this issue May 7, 2014
@spinscale
Removed Index Status API
0c0f717 
The functionality of the index status API has been replaced by the recovery API.

Relates elastic#4854
spinscale added a commit that referenced this issue May 7, 2014
@spinscale
Deprecate Index Status API
f0d73a8 
Elasticsearch 1.1 introduced the recovery API, which replaces the
functionality of the index status API.
This marks the index status API as deprecated in order to remove it in the future.

Relates #4854
@spinscale spinscale added doc and removed v2.0.0 labels May 7, 2014
@spinscale
Copy link
Contributor

closed by f0d73a8

@karmi karmi mentioned this issue Nov 4, 2015
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@spinscale spinscale

Labels
v1.2.0
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@clintongormley @kevinkluge @spinscale @s1monw