[Rename] Rename files under "qa" folder #194

harold-wang · 2021-03-04T00:40:21Z

Issue #160, if available:

Description of changes:

Rename files under "qa" folder:

Rename Elasticsearch* class to Opensearch*
Rename elasticsearch* instance to opensearch*
Rename package org.elasticsearch.* to org.opensearch.*
Rename import org.elasticsearch.* to org.opensearch.*

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

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

odfe-release-bot · 2021-03-04T02:45:03Z

Request for Admin to accept this test.

nknize

A couple comments but otherwise LGTM.

qa/die-with-dignity/src/main/java/org/elasticsearch/DieWithDignityPlugin.java

qa/die-with-dignity/src/main/java/org/elasticsearch/RestDieWithDignityAction.java

nknize · 2021-03-04T06:37:51Z

...de-name/src/test/java/org/elasticsearch/unconfigured_node_name/JsonLogsFormatAndParseIT.java

@@ -36,7 +36,7 @@
    private static final String OS_NAME = System.getProperty("os.name");
    private static final boolean WINDOWS = OS_NAME.startsWith("Windows");

-    // These match the values defined in org.elasticsearch.gradle.testclusters.ElasticsearchNode
+    // These match the values defined in org.opensearch.gradle.testclusters.OpensearchNode


'S' in OpenSearch should be capital OpenSearchNode

Thanks @nknize it was ElasticsearchNode, I think we should use "OpensearchNode" instead of "OpenSearchNode". I will merge, will keep a note to check if I see "org.elasticsearch.gradle.testclusters.OpenSearchNode" after all renaming is done.

@nknize I changed OpensearchNode to OpenSearchNode in PR #212, thanks

adnapibar · 2021-03-04T07:50:06Z

...-clusters/src/test/java/org/elasticsearch/search/CrossClusterSearchUnavailableClusterIT.java

@@ -17,47 +17,47 @@
 * under the License.
 */

-package org.elasticsearch.search;
+package org.opensearch.search;


file directory doesn't match with package name, please rename those.

I put all the git mv to different PR. thanks

adnapibar · 2021-03-04T07:56:02Z

qa/logging-config/src/test/java/org/elasticsearch/common/logging/JsonLogsTestSetup.java

@@ -16,7 +16,7 @@
 * specific language governing permissions and limitations
 * under the License.
 */
-package org.elasticsearch.common.logging;
+package org.opensearch.common.logging;


directory name doesn't match with package name

I put all the git mv to different PR. thanks as I mentioned #194 (comment)

adnapibar · 2021-03-04T07:56:18Z

qa/full-cluster-restart/src/test/java/org/elasticsearch/upgrades/FullClusterRestartIT.java

@@ -17,34 +17,34 @@
 * under the License.
 */

-package org.elasticsearch.upgrades;
+package org.opensearch.upgrades;


directory name doesn't match with package name

I put all the git mv to different PR. thanks as I mentioned in #194 (comment)

adnapibar · 2021-03-04T07:56:54Z

qa/evil-tests/src/test/java/org/elasticsearch/bootstrap/ESPolicyUnitTests.java

@@ -17,10 +17,10 @@
 * under the License.
 */

-package org.elasticsearch.bootstrap;
+package org.opensearch.bootstrap;


file directory doesn't match with package name.

file name needs to be changed to OpenSearchPolicyUnitTests.java

I put all the git mv to different PR as I mentioned #194 (comment)

adnapibar · 2021-03-04T07:57:19Z

qa/evil-tests/src/test/java/org/elasticsearch/bootstrap/EvilElasticsearchCliTests.java

@@ -17,13 +17,13 @@
 * under the License.


file name needs to be changed

I put all the git mv to different PR as I mentioned in #194 (comment)

adnapibar · 2021-03-04T08:02:05Z

qa/rolling-upgrade/src/test/java/org/elasticsearch/upgrades/AbstractRollingTestCase.java

@@ -16,11 +16,11 @@
 * specific language governing permissions and limitations
 * under the License.
 */
-package org.elasticsearch.upgrades;
+package org.opensearch.upgrades;


directory name doesn't match with package name

file or dir rename is taken care of in another PR.

adnapibar · 2021-03-04T08:02:49Z

qa/smoke-test-http/src/test/java/org/elasticsearch/http/ContextAndHeaderTransportIT.java

@@ -17,46 +17,46 @@
 * under the License.
 */

-package org.elasticsearch.http;
+package org.opensearch.http;


directory name doesn't match with package name

file or dir rename is taken care of in another PR.

adnapibar · 2021-03-04T08:02:54Z

qa/smoke-test-client/src/test/java/org/elasticsearch/smoketest/SmokeTestClientIT.java

@@ -17,11 +17,11 @@
 * under the License.
 */

-package org.elasticsearch.smoketest;
+package org.opensearch.smoketest;


directory name doesn't match with package name

file or dir rename is taken care of in another PR.

adnapibar · 2021-03-04T08:03:44Z

.../src/test/java/org/elasticsearch/smoketest/SmokeTestIngestDisabledClientYamlTestSuiteIT.java

@@ -17,13 +17,13 @@
 * under the License.
 */

-package org.elasticsearch.smoketest;
+package org.opensearch.smoketest;


directory name doesn't match with package name

file or dir rename is taken care of in another PR.

adnapibar · 2021-03-04T08:03:51Z

...est-with-all-dependencies/src/test/java/org/elasticsearch/ingest/AbstractScriptTestCase.java

-import org.elasticsearch.script.TemplateScript;
-import org.elasticsearch.script.mustache.MustacheScriptEngine;
-import org.elasticsearch.test.ESTestCase;
+package org.opensearch.ingest;


directory name doesn't match with package name

file or dir rename is taken care of in another PR.

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

* [Rename] Rename qa folder Signed-off-by: Harold Wang <harowang@amazon.com> * Remove the dot in the end of "package org.opensearch." Signed-off-by: Harold Wang <harowang@amazon.com> * Add semicolon Signed-off-by: Harold Wang <harowang@amazon.com>

Signed-off-by: dblock <dblock@amazon.com>

[Rename] Rename qa folder

b2eb324

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

harold-wang requested review from abbashus, adnapibar and nknize March 4, 2021 00:41

harold-wang self-assigned this Mar 4, 2021

harold-wang added Rename Renaming to OpenSearch >FORK Related to the fork process labels Mar 4, 2021

harold-wang changed the base branch from main to rename/opensearch March 4, 2021 00:43

nknize approved these changes Mar 4, 2021

View reviewed changes

adnapibar suggested changes Mar 4, 2021

View reviewed changes

harold-wang added 2 commits March 4, 2021 06:45

Remove the dot in the end of "package org.opensearch."

d3d66ee

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

Add semicolon

7cd3785

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

harold-wang merged commit fb13600 into opensearch-project:rename/opensearch Mar 4, 2021

harold-wang mentioned this pull request Mar 8, 2021

Fork rename to opensearch #171

Closed

jamesiri unassigned harold-wang Mar 15, 2021

nknize mentioned this pull request Mar 22, 2021

[Rename] Rename from org.elasticsearch to org.opensearch #160

Closed

ritty27 pushed a commit to ritty27/OpenSearch that referenced this pull request May 12, 2024

Removed obsolete publishForReleaseManager task. (opensearch-project#194)

103ff25

Signed-off-by: dblock <dblock@amazon.com>

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Rename] Rename files under "qa" folder #194

[Rename] Rename files under "qa" folder #194

harold-wang commented Mar 4, 2021 •

edited

Loading

odfe-release-bot commented Mar 4, 2021

nknize left a comment

nknize Mar 4, 2021

harold-wang Mar 4, 2021 •

edited

Loading

harold-wang Mar 5, 2021

adnapibar Mar 4, 2021

harold-wang Mar 4, 2021

adnapibar Mar 4, 2021

harold-wang Mar 4, 2021

adnapibar Mar 4, 2021

harold-wang Mar 4, 2021

adnapibar Mar 4, 2021

harold-wang Mar 4, 2021

adnapibar Mar 4, 2021

harold-wang Mar 4, 2021

adnapibar Mar 4, 2021

harold-wang Mar 4, 2021

adnapibar Mar 4, 2021

harold-wang Mar 4, 2021

adnapibar Mar 4, 2021

harold-wang Mar 4, 2021

adnapibar Mar 4, 2021

harold-wang Mar 4, 2021

adnapibar Mar 4, 2021

harold-wang Mar 4, 2021

[Rename] Rename files under "qa" folder #194

[Rename] Rename files under "qa" folder #194

Conversation

harold-wang commented Mar 4, 2021 • edited Loading

odfe-release-bot commented Mar 4, 2021

nknize left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

harold-wang Mar 4, 2021 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

harold-wang commented Mar 4, 2021 •

edited

Loading

harold-wang Mar 4, 2021 •

edited

Loading