-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[Rename] ElasticsearchParseException class in server module #169
[Rename] ElasticsearchParseException class in server module #169
Conversation
This commit refactors ElasticsearchParseException class in the server module to OpenSearchParseException. References and usages throughout the rest of the codebase are fully refactored. Signed-off-by: Nicholas Knize <[email protected]>
…eException Signed-off-by: Nicholas Knize <[email protected]>
Signed-off-by: Nicholas Knize <[email protected]>
Request for Admin to accept this test. |
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.
Couple of comments - looks good otherwise 👍
...est-common/src/internalClusterTest/java/org/elasticsearch/ingest/common/IngestRestartIT.java
Outdated
Show resolved
Hide resolved
@@ -799,8 +799,8 @@ public String toString() { | |||
org.elasticsearch.indices.IndexPrimaryShardNotAllocatedException::new, 33, UNKNOWN_VERSION_ADDED), | |||
TRANSPORT_EXCEPTION(org.elasticsearch.transport.TransportException.class, | |||
org.elasticsearch.transport.TransportException::new, 34, UNKNOWN_VERSION_ADDED), | |||
ELASTICSEARCH_PARSE_EXCEPTION(org.elasticsearch.ElasticsearchParseException.class, | |||
org.elasticsearch.ElasticsearchParseException::new, 35, UNKNOWN_VERSION_ADDED), | |||
ELASTICSEARCH_PARSE_EXCEPTION(org.elasticsearch.OpenSearchParseException.class, |
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.
Are we renaming the enums as part of a separate PR?
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.
Yeah, I think at this point making a separate sweeping rename PR is the right move
Signed-off-by: Nicholas Knize <[email protected]>
This commit refactors ElasticsearchParseException class in the server module to OpenSearchParseException. References and usages throughout the rest of the codebase are fully refactored. Signed-off-by: Nicholas Knize <[email protected]>
…project#169) Signed-off-by: Rene Cordier <[email protected]>
* Sort field in Hit deserializer should handle null events (opensearch-project#169) Signed-off-by: Rene Cordier <[email protected]> Signed-off-by: Matt Timmermans <[email protected]> * Deprecating master terminology to support inclusive naming (opensearch-project#167) * Deprecating master terminology to support inclusive naming Signed-off-by: Vacha Shah <[email protected]> * Fixing tests against unreleased OpenSearch Signed-off-by: Vacha Shah <[email protected]> Signed-off-by: Matt Timmermans <[email protected]> * Fix issue 172 Signed-off-by: Matt Timmermans <[email protected]> * Add test for issue 172 Signed-off-by: Matt Timmermans <[email protected]> Co-authored-by: Rene Cordier <[email protected]> Co-authored-by: Vacha Shah <[email protected]> Co-authored-by: Matt Timmermans <[email protected]>
This PR refactors ElasticsearchParseException class in the server module to
OpenSearchParseException. References and usages throughout the rest of the
codebase are fully refactored.
relates #160