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
Introducing VersionType.FORCE & VersionType.EXTERNAL_GTE #4993
Conversation
Also added "external_gt" as an alias name for VersionType.EXTERNAL , accessible for the rest layer. Closes elastic#4213 , Closes elastic#2946
…ue where routing value is required and version type is not internal.
@@ -328,6 +335,7 @@ protected ShardIterator shards(ClusterState clusterState, BulkShardRequest reque | |||
} | |||
|
|||
assert responses[requestIndex] != null; // we must have set a response somewhere. | |||
assert preVersionTypes[requestIndex] != null; |
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.
should we move this up and fail earlier?
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.
My only reason to add it was to make sure that none of the code paths forget to set it. Did it when I found a place that did (DeleteRequest)
I think this looks good - I still would want @kimchy to look at it before we push |
LGTM! |
I don't find this feature in the docs, especially not within the "optimistic concurrency control" documentation where i'd expect it ... is this still alive? |
@mrkamel very much alive and described here: http://www.elastic.co/guide/en/elasticsearch/reference/current/docs-index_.html#_version_types |
Also added "external_gt" as an alias name for VersionType.EXTERNAL , accessible for the rest layer.
Closes #4213 , Closes #2946