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

Change IndexPrimaryShardNotAllocatedException from 409 to 500 #7987

Closed
wants to merge 1 commit into from

Conversation

reuben-sutton
Copy link
Contributor

Closes #7632

Change IndexPrimaryShardNotAllocatedException from 409 (RestStatus.CONFLICT) to 500 (RestStatus.INTERNAL_SERVER_ERROR)

Change IndexPrimaryShardNotAllocatedException from 409 (RestStatus.CONFLICT) to 500 (RestStatus.INTERNAL_SERVER_ERROR)
@reuben-sutton
Copy link
Contributor Author

@clintongormley could you review please?

@clintongormley
Copy link

I've labelled this for review - my biggest concern is whether any tests start failing? @bleskes could you take a look at this please?

@bleskes
Copy link
Contributor

bleskes commented Oct 16, 2014

Test pass and it looks good to me. We can consider in a future change whether to remove all together (or use it more) - it's hardly used.

@reuben-sutton can I ask you to sign our CLA so I can merge it in: http://www.elasticsearch.org/contributor-agreement/

@reuben-sutton
Copy link
Contributor Author

@bleskes bleskes closed this in 645ce26 Oct 20, 2014
bleskes pushed a commit that referenced this pull request Oct 20, 2014
…NFLICT) to 500 (RestStatus.INTERNAL_SERVER_ERROR)

Closes #7632, Closes #7987
@bleskes
Copy link
Contributor

bleskes commented Oct 20, 2014

@reuben-sutton merged. Thx!

@jpountz jpountz removed the review label Oct 21, 2014
@clintongormley clintongormley added :Exceptions and removed :Core/Infra/REST API REST infrastructure and utilities labels Jun 7, 2015
@lcawl lcawl added :Core/Infra/Core Core issues without another label and removed :Exceptions labels Feb 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Change IndexPrimaryShardNotAllocatedException from 409 to 500
6 participants