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

[Rename] plugins #193

Merged
merged 17 commits into from
Mar 11, 2021
Merged

[Rename] plugins #193

merged 17 commits into from
Mar 11, 2021

Conversation

harold-wang
Copy link
Contributor

@harold-wang harold-wang commented Mar 4, 2021

Issue #160 , if available:

Description of changes:

This PR refactors files under "plugins" folder s part of the Elasticsearch to OpenSearch renaming effort.

Signed-off-by: Harold Wang harowang@amazon.com

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

… plugin folder, elasticsearch package name is also refactored.

Signed-off-by: Harold Wang <harowang@amazon.com>
@harold-wang harold-wang changed the base branch from main to rename/opensearch March 4, 2021 00:15
@harold-wang harold-wang changed the title [Rename] Refector plugins folder [Rename] Rename plugins folder Mar 4, 2021
@harold-wang harold-wang self-assigned this Mar 4, 2021
@harold-wang harold-wang added Rename Renaming to OpenSearch >FORK Related to the fork process labels Mar 4, 2021
@harold-wang harold-wang changed the title [Rename] Rename plugins folder [Rename] Rename files under plugins folder Mar 4, 2021
@odfe-release-bot
Copy link

Request for Admin to accept this test.

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.

Please refactor the directory names along with the package names - you can use IntelliJ for renaming.

@harold-wang
Copy link
Contributor Author

harold-wang commented Mar 4, 2021

Please refactor the directory names along with the package names - you can use IntelliJ for renaming.

Done, thx.

@adnapibar
Copy link
Contributor

adnapibar commented Mar 4, 2021

Please refactor the directory names along with the package names - you can use IntelliJ for renaming.

I will put all git rename in another PR, thx.

I would say the renaming is done as part of the same PR - I don't see the point doing it separately.

Agree, it was done in same PR. thanks

@harold-wang
Copy link
Contributor Author

harold-wang commented Mar 4, 2021

Please refactor the directory names along with the package names - you can use IntelliJ for renaming.

I will put the filename and directory rename in another PR, thanks
I take this suggestion, done it in the same PR.

@harold-wang
Copy link
Contributor Author

Please refactor the directory names along with the package names - you can use IntelliJ for renaming.

I will put all git rename in another PR, thx.

I would say the renaming is done as part of the same PR - I don't see the point doing it separately.

OK, I take this suggestions, thanks.

Signed-off-by: Harold Wang <harowang@amazon.com>
Signed-off-by: Harold Wang <harowang@amazon.com>
Rename ESSingleNodeTestCase to OpenSearchSingleNodeTestCase

Signed-off-by: Harold Wang <harowang@amazon.com>
Signed-off-by: Harold Wang <harowang@amazon.com>
Signed-off-by: Harold Wang <harowang@amazon.com>
Rename elasticsearch@BUILD.ELASTICO.CO" to "opensearch@BUILD.OPENSEARCH.CO"

Signed-off-by: Harold Wang <harowang@amazon.com>
Signed-off-by: Harold Wang <harowang@amazon.com>
Rename elasticsearch to opensearch

Signed-off-by: Harold Wang <harowang@amazon.com>
Signed-off-by: Harold Wang <harowang@amazon.com>
Signed-off-by: Harold Wang <harowang@amazon.com>
Rename elasticsearch to opensearch

Signed-off-by: Harold Wang <harowang@amazon.com>
@harold-wang
Copy link
Contributor Author

@adnapibar @nknize @abbashus could you please help me review this one? thanks

@harold-wang
Copy link
Contributor Author

Please refactor the directory names along with the package names - you can use IntelliJ for renaming.
Done

@harold-wang
Copy link
Contributor Author

Please refactor the directory names along with the package names - you can use IntelliJ for renaming.

I will put all git rename in another PR, thx.

Done, thanks

Signed-off-by: Harold Wang <harowang@amazon.com>
…search/search into rename_plugin

Signed-off-by: Harold Wang <harowang@amazon.com>
"zone": "europe-west1-b"
}
]
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Just notice these 3 files with name instances are not identified as renaming, please verify that git can find the files' history.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Do not worry, these file is not part of version controlled file.
git mv elasticsearch opensearch
fatal: source directory is empty, source=plugins/discovery-gce/out/test/resources/org/elasticsearch, destination=plugins/discovery-gce/out/test/resources/org/opensearch

Copy link
Collaborator

Choose a reason for hiding this comment

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

I see. Thanks for telling me! 😄

Signed-off-by: Harold Wang <harowang@amazon.com>
Copy link
Collaborator

@tlfeng tlfeng left a comment

Choose a reason for hiding this comment

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

Thank you! All my comments above has been resolved.

@harold-wang harold-wang changed the title [Rename] Rename files under plugins folder [Rename] plugins Mar 11, 2021
@harold-wang harold-wang merged commit f27f5ac into opensearch-project:rename/opensearch Mar 11, 2021
nknize pushed a commit that referenced this pull request Mar 22, 2021
* [Rename] plugins (#193)

This PR refactors files under "plugins" folders part of the Elasticsearch to OpenSearch renaming effort.

Signed-off-by: Harold Wang <harowang@amazon.com>
ritty27 pushed a commit to ritty27/OpenSearch that referenced this pull request May 12, 2024
Signed-off-by: dblock <dblock@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>FORK Related to the fork process Rename Renaming to OpenSearch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants