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.

Sign up for GitHub

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

Bulk scripts should also support retry_on_conflict #628

Closed
nik9000 opened this issue Jun 5, 2014 · 2 comments · Fixed by #629
Closed

Bulk scripts should also support retry_on_conflict #628

nik9000 opened this issue Jun 5, 2014 · 2 comments · Fixed by #629

Comments

@nik9000
Copy link
Contributor

nik9000 commented Jun 5, 2014

#623 but for scripts.
nik9000 pushed a commit to nik9000/Elastica that referenced this issue Jun 5, 2014
Merge some of update by document and script
6cff6b2 
These share many of the same options so it makes sense to merge them.

Closes ruflin#628
Closes ruflin#627
@ruflin
Copy link
Owner

ruflin commented Jun 5, 2014

Do you already plan a pull request? :-)

@nik9000
Copy link
Contributor Author

nik9000 commented Jun 5, 2014

Sorry, yeah, I'm just letting travis run the regression tests first. Same for #627. Well, they kind of get fixed together because doing that one gets this one "for free".

@nik9000 nik9000 mentioned this issue Jun 5, 2014
Merged
nik9000 pushed a commit to nik9000/Elastica that referenced this issue Jun 5, 2014
Merge some of update by document and script
e8f2047 
These share many of the same options so it makes sense to merge them.

Closes ruflin#628
Closes ruflin#627
nik9000 pushed a commit to nik9000/Elastica that referenced this issue Jun 11, 2014
Merge some of update by document and script
8c11f52 
These share many of the same options so it makes sense to merge them.

Closes ruflin#628
Closes ruflin#627
nik9000 pushed a commit to nik9000/Elastica that referenced this issue Jun 11, 2014
Merge some of update by document and script
5b60b70 
These share many of the same options so it makes sense to merge them.
This also adds support for things like _retry_on_conflict and _routing to
updates by scripts.

Closes ruflin#628
Closes ruflin#627
nik9000 pushed a commit to nik9000/Elastica that referenced this issue Jun 12, 2014
Merge some of update by document and script
1b16215 
These share many of the same options so it makes sense to merge them.
This also adds support for things like _retry_on_conflict and _routing to
updates by scripts.

Closes ruflin#628
Closes ruflin#627
nik9000 pushed a commit to nik9000/Elastica that referenced this issue Jun 13, 2014
Merge some of update by document and script
23ab5ee 
These share many of the same options so it makes sense to merge them.
This also adds support for things like _retry_on_conflict and _routing to
updates by scripts.

Closes ruflin#628
Closes ruflin#627
nik9000 pushed a commit to nik9000/Elastica that referenced this issue Jun 13, 2014
Merge some of update by document and script
8e2cafe 
These share many of the same options so it makes sense to merge them.
This also adds support for things like _retry_on_conflict and _routing to
updates by scripts.

Closes ruflin#628
Closes ruflin#627
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Merge some of update by document and script nik9000/Elastica
2 participants
@nik9000 @ruflin