-
Notifications
You must be signed in to change notification settings - Fork 24.6k
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
Add a REST spec for the create API #20924
Conversation
The create request now requires that an ID be present. Currently the clients hard code a create method, but we should just add a create REST spec so this method can be autogenerated.
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.
LGTM. Left a minor question
{ | ||
"create": { | ||
"documentation": "http://www.elastic.co/guide/en/elasticsearch/reference/master/docs-index_.html", | ||
"methods": ["PUT"], |
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.
we also except POST here in the java code. Not sure what the convention is though - do we want to be "correct" or reflect what he java does?
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.
I'll add POST, thanks for the review
/cc @elastic/es-clients |
The create request now requires that an ID be present. Currently the clients hard code a create method, but we should just add a create REST spec so this method can be autogenerated.
The create request now requires that an ID be present. Currently the clients hard code a create method, but we should just add a create REST spec so this method can be autogenerated.
Now that the create api has its own spec, we can remove the special case in the yaml test client for it Relates to elastic#20924
Now that the create api has its own spec, we can remove the special case in the yaml test client for it Relates to #20924
Now that the create api has its own spec, we can remove the special case in the yaml test client for it Relates to #20924
Now that the create api has its own spec, we can remove the special case in the yaml test client for it Relates to #20924
The create request now requires that an ID be present.
Currently the clients hard code a create method, but
we should just add a create REST spec so this method
can be autogenerated.