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

Removed duplicate timeout param #10205

Closed
wants to merge 1 commit into from

Conversation

timschlechter
Copy link
Contributor

Minor change; I removed the duplicate timeout param in the bulk.json rest-api-spec. It causes errors when deserialising the params to a map.

@timschlechter
Copy link
Contributor Author

Oops, I submitted the PR before signing the CLA.
I've signed it now, do I need to take any action on the PR?

@javanna
Copy link
Member

javanna commented Mar 21, 2015

All good @timschlechter thanks a lot for your PR, will merge it soon.

@javanna javanna self-assigned this Mar 21, 2015
@timschlechter
Copy link
Contributor Author

Ok, thanks!

I noticed the duplicate exists in several release branches, it was introduced more than ayear ago in e613ecf

Would you like me to create a PR for each branch, or could someone with commit rights fix it directly in the other branches?

@javanna
Copy link
Member

javanna commented Mar 22, 2015

Merged, thanks! I also backported it to 1.x, 1.5, 1.4 and 1.3 branches.

@timschlechter
Copy link
Contributor Author

Great, thank you for the quick merge and fixes!

mute pushed a commit to mute/elasticsearch that referenced this pull request Jul 29, 2015
mute pushed a commit to mute/elasticsearch that referenced this pull request Jul 29, 2015
@lcawl lcawl added :Distributed/CRUD A catch all label for issues around indexing, updating and getting a doc by id. Not search. and removed :Bulk labels Feb 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug :Distributed/CRUD A catch all label for issues around indexing, updating and getting a doc by id. Not search. v1.3.10 v1.4.5 v1.5.0 v1.6.0 v2.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants