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 status
code from main action /
#8865
Remove status
code from main action /
#8865
Conversation
Wondering if it should be listed in breaking changes? |
@@ -72,7 +72,6 @@ public void handleRequest(final RestRequest request, RestChannel channel, final | |||
} | |||
|
|||
builder.startObject(); | |||
builder.field("status", status.getStatus()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a JAVA API equivalent to this? if so we should update that too so the two are consistent with each other?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@colings86 I did not find any Java equivalent to GET /
.
LGTM |
@clintongormley Do you think this should be listed as a breaking change? |
@dadoonet yes |
683f161
to
c0c5c26
Compare
@colings86 @clintongormley I added a section in |
c0c5c26
to
237ef3e
Compare
Today we give the HTTP status back within the HTTP response itself and within the JSON response as well: ```sh curl localhost:9200/ ``` ```js { "status" : 200, "name" : "Red Wolf", "version" : { "number" : "2.0.0", "build_hash" : "6837a61d8a646a2ac7dc8da1ab3c4ab85d60882d", "build_timestamp" : "2014-08-19T13:55:56Z", "build_snapshot" : true, "lucene_version" : "4.9" }, "tagline" : "You Know, for Search" } ```
237ef3e
to
e2bcef7
Compare
Merged with 0526459 |
status
code from main action /
Today we give the HTTP status back within the HTTP response itself and within the JSON response as well:
We can remove the
status
in elasticsearch 2.0.