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

Make ElasticsearchException ids for 2.0 consistent with master #13805

Merged
merged 1 commit into from Sep 25, 2015

Conversation

Projects
None yet
4 participants
@jasontedor
Member

jasontedor commented Sep 25, 2015

This commit reworks the ElasticsearchException ids so that they are
consistent with master at f40ae25.

@s1monw

View changes

Show outdated Hide outdated core/src/test/java/org/elasticsearch/ExceptionSerializationTests.java
@s1monw

This comment has been minimized.

Show comment
Hide comment
@s1monw

s1monw Sep 25, 2015

Contributor

left one comment

Contributor

s1monw commented Sep 25, 2015

left one comment

Make ElasticsearchException ids for 2.0 consistent with master
This commit reworks the ElasticsearchException ids so that they are
consistent with master at f40ae25.
@s1monw

This comment has been minimized.

Show comment
Hide comment
@s1monw

s1monw Sep 25, 2015

Contributor

LGTM

Contributor

s1monw commented Sep 25, 2015

LGTM

jasontedor added a commit that referenced this pull request Sep 25, 2015

Merge pull request #13805 from jasontedor/exception-ids-consistent-wi…
…th-master

Make ElasticsearchException ids for 2.0 consistent with master

@jasontedor jasontedor merged commit 2295d13 into elastic:2.0 Sep 25, 2015

1 check passed

CLA Commit author is a member of Elasticsearch
Details

@jasontedor jasontedor deleted the jasontedor:exception-ids-consistent-with-master branch Sep 25, 2015

@jasontedor jasontedor removed the review label Sep 25, 2015

@clintongormley clintongormley added v2.0.0-rc1 and removed v2.0.0 labels Oct 7, 2015

@lcawl lcawl added :Core/Core 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