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

[Rename] server/src/test/java/org/elasticsearch/<files-only> #188

Merged
merged 11 commits into from
Mar 4, 2021
Merged

[Rename] server/src/test/java/org/elasticsearch/<files-only> #188

merged 11 commits into from
Mar 4, 2021

Conversation

abbashus
Copy link
Contributor

@abbashus abbashus commented Mar 3, 2021

This PR refactors all instances of elasticsearch in server/src/test/java/org/elasticsearch/ to opensearch.

  • Change package from org.elasticsearch to org.opensearch
  • Change class reference from ElasticsearchException to OpenSearchException
  • Change test method name containing Elasticsearch to Opensearch
  • Refactored Build.getElasticsearchCodeSourceLocation() to Build.getOpenSearchCodeSourceLocation() and their usage in test

relates #160

@abbashus
Refactor package org.elasticsearch to org.opensearch
aafe4b8 
Signed-off-by: Abbas Hussain <abbas_10690@yahoo.com>
@abbashus
Refactor imports from org.elasticsearch to org.opensearch
d16189d 
Signed-off-by: Abbas Hussain <abbas_10690@yahoo.com>
@abbashus
Rename class with *Elasticsearch* to *OpenSearch*
c6c1a71 
Signed-off-by: Abbas Hussain <abbas_10690@yahoo.com>
@abbashus
Refactor instances of ElasticSearchExcpetion to OpenSearchException
276a544 
Signed-off-by: Abbas Hussain <abbas_10690@yahoo.com>
@abbashus
Refactor instances of ElasticsearchSecurityException to OpenSearchSec…
3e8d5f6 
…urityException

Signed-off-by: Abbas Hussain <abbas_10690@yahoo.com>
@abbashus
- Refactor instances of *Elasticsearch*Exception
28addff 
- Rename methods with Elasticsearch to OpenSearch and their usage
- Refactored Build.getElasticsearchCodeSourceLocation() to Build.getOpenSearchCodeSourceLocation()

Signed-off-by: Abbas Hussain <abbas_10690@yahoo.com>
@abbashus
Update string with elasticsearch to opensearch
4a8ca08 
Signed-off-by: Abbas Hussain <abbas_10690@yahoo.com>
@abbashus abbashus self-assigned this Mar 3, 2021
@abbashus abbashus added >FORK Related to the fork process Rename Renaming to OpenSearch labels Mar 3, 2021
@abbashus
Merge branch 'rename/opensearch' into refactor-test-elasticsearch
c321187 
Signed-off-by: Abbas Hussain <abbas_10690@yahoo.com>
@abbashus abbashus mentioned this pull request Mar 3, 2021
Closed
@abbashus
Remove extra line
eb3aa01 
Signed-off-by: Abbas Hussain <abbas_10690@yahoo.com>
@odfe-release-bot
Copy link

Request for Admin to accept this test.

adnapibar
adnapibar approved these changes Mar 4, 2021
View reviewed changes
Copy link
Contributor

@adnapibar adnapibar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor comment, otherwise looks good

server/src/test/java/org/opensearch/OpenSearchExceptionTests.java Outdated Show resolved Hide resolved
@abbashus
Resolve merge conflicts
8a97445 
Signed-off-by: Abbas Hussain <abbas_10690@yahoo.com>
@abbashus
Rename string es abbrev to opensearch
025de44 
Signed-off-by: Abbas Hussain <abbas_10690@yahoo.com>
@abbashus abbashus merged commit fa2a229 into opensearch-project:rename/opensearch Mar 4, 2021
nknize pushed a commit that referenced this pull request Mar 22, 2021
@abbashus @nknize
[Rename] server/src/test/java/org/elasticsearch/<files-only> (#188)
2b8a5d0 
Signed-off-by: Abbas Hussain <abbas_10690@yahoo.com>
ritty27 pushed a commit to ritty27/OpenSearch that referenced this pull request May 12, 2024
@gaiksaya
Add automatic triggers to jenkins workflow (opensearch-project#188)
a1c6085 
* Add triggers

Signed-off-by: Sayali Gaikawad <gaiksaya@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adnapibar adnapibar adnapibar approved these changes

@CEHENKLE CEHENKLE Awaiting requested review from CEHENKLE

@harold-wang harold-wang Awaiting requested review from harold-wang

@nknize nknize Awaiting requested review from nknize

Assignees
No one assigned
Labels
>FORK Related to the fork process Rename Renaming to OpenSearch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@abbashus @odfe-release-bot @adnapibar