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 Restart API and remove Node#stop() #9921

Merged
merged 1 commit into from Feb 27, 2015

Conversation

@s1monw
Copy link
Contributor

commented Feb 27, 2015

The restart API seems pretty esoteric and is entirely untested.
This commit removes it without a replacement.

Closes #9841

The restart API seems pretty esoteric and is entirely untested.
This commit removes it without a replacement.

Closes #9841
@s1monw s1monw force-pushed the s1monw:remove_restart branch to ce53e20 Feb 27, 2015
@s1monw

This comment has been minimized.

Copy link
Contributor Author

commented Feb 27, 2015

btw. this is entirely undocumented and disabled by default. I don't think we need to maintain it at all. It's also entirely untested...

@clintongormley

This comment has been minimized.

Copy link
Member

commented Feb 27, 2015

+1

@s1monw s1monw merged commit ce53e20 into elastic:master Feb 27, 2015
1 check passed
1 check passed
CLA Commit author is a member of Elasticsearch
Details
@s1monw s1monw deleted the s1monw:remove_restart branch Feb 27, 2015
@clintongormley clintongormley removed the review label Mar 19, 2015
@clintongormley clintongormley changed the title [CORE] Remove Restart API and remove Node#stop() Remove Restart API and remove Node#stop() Jun 6, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.