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

Add a created flag to IndexResponse & UpdateResponse #3154

Closed
bleskes opened this issue Jun 9, 2013 · 2 comments
Closed

Add a created flag to IndexResponse & UpdateResponse #3154

bleskes opened this issue Jun 9, 2013 · 2 comments
Assignees
@bleskes
Labels
>feature v1.0.0.Beta1

Comments

@bleskes
Copy link
Contributor

bleskes commented Jun 9, 2013

The new flag indicates whether the operation added the document to the index (true) or replaced an existing one (false).
@ghost ghost assigned bleskes Jun 9, 2013
This was referenced Jun 9, 2013
Closed
Closed
bleskes added a commit to bleskes/elasticsearch that referenced this issue Jun 12, 2013
@bleskes
Added created flag to index related request classes.
0aaa383 
The flag is set to true when a document is new, false when replacing an existing object.

Other minor changes:
Fixed an issue with dynamic gc deletes settings update
Added an assertThrows to ElasticsearchAssertion

Closes elastic#3084 , Closes elastic#3154
@synhershko
Copy link
Contributor

Any chance to push this to 0.90 as well?

@bleskes
Copy link
Contributor Author

bleskes commented Jun 13, 2013

That's tricky because it breaks backward compatibility on the wire level (internal communication protocols between nodes), which needs a major version change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bleskes bleskes

Labels
>feature v1.0.0.Beta1
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Added created flag to index related request classes. bleskes/elasticsearch
2 participants
@synhershko @bleskes