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] refactor o.e.action.admin.cluster #207

Merged
merged 1 commit into from
Mar 9, 2021
Merged

[Rename] refactor o.e.action.admin.cluster #207

merged 1 commit into from
Mar 9, 2021

Conversation

nknize
Copy link
Collaborator

@nknize nknize commented Mar 4, 2021 •
edited
Loading

This PR refactors all classes and directories under
o.e.action.admin.cluster to o.opensearch.action.admin.cluster. References
throughout the codebase are updated to reflect the refactor.

relates #160

@nknize nknize added >FORK Related to the fork process Rename Renaming to OpenSearch labels Mar 4, 2021
@odfe-release-bot
Copy link

Request for Admin to accept this test.

@nknize nknize mentioned this pull request Mar 4, 2021
Closed
saratvemulapalli
saratvemulapalli reviewed Mar 8, 2021
View reviewed changes
Copy link
Member

@saratvemulapalli saratvemulapalli left a comment

Choose a reason for hiding this comment

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

Overall the changes look good to me.
Some minor comments.

There are few white space changes for ALv2 headers, not sure if they are intentionally added.
Dropped comments in few places, but there are few more.

...rTest/java/org/elasticsearch/action/admin/cluster/allocation/ClusterAllocationExplainIT.java Outdated Show resolved Hide resolved
...va/org/elasticsearch/action/admin/cluster/state/TransportClusterStateActionDisruptionIT.java Show resolved Hide resolved
...rc/internalClusterTest/java/org/elasticsearch/action/admin/cluster/stats/ClusterStatsIT.java Show resolved Hide resolved
server/src/main/java/org/elasticsearch/cluster/metadata/RepositoriesMetadata.java Show resolved Hide resolved
server/src/main/java/org/elasticsearch/snapshots/RestoreInfo.java Show resolved Hide resolved
...a/org/opensearch/action/admin/cluster/allocation/ClusterAllocationExplainRequestBuilder.java Show resolved Hide resolved
...in/java/org/opensearch/action/admin/cluster/allocation/ClusterAllocationExplainResponse.java Show resolved Hide resolved
...c/main/java/org/opensearch/action/admin/cluster/allocation/ClusterAllocationExplanation.java Show resolved Hide resolved
.../org/opensearch/action/admin/cluster/allocation/TransportClusterAllocationExplainAction.java Show resolved Hide resolved
.../java/org/opensearch/action/admin/cluster/configuration/AddVotingConfigExclusionsAction.java Show resolved Hide resolved
@nknize nknize force-pushed the rename/server/action_admin_cluster branch from 0fb8996 to b14575b Compare March 9, 2021 05:25
@nknize
[Rename] merge conflict resolution
cad45b9 
Signed-off-by: Nicholas Knize <nknize@amazon.com>
@nknize nknize force-pushed the rename/server/action_admin_cluster branch from b14575b to cad45b9 Compare March 9, 2021 22:53
@nknize nknize merged commit 23dc261 into opensearch-project:rename/opensearch Mar 9, 2021
nknize added a commit that referenced this pull request Mar 22, 2021
@nknize
[Rename] refactor o.e.action.admin.cluster (#207)
8aa818e 
This commit refactors all classes in o.e.action.admin.cluster to 
org.opensearch.action.admin.cluster. References are updated 
throughout the codebase.

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

@saratvemulapalli saratvemulapalli saratvemulapalli approved these changes

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
@nknize @odfe-release-bot @saratvemulapalli