Match existing naming scheme

Signed-off-by: Thomas Farr <[email protected]>
Xtansia · Feb 27, 2024 · f7cde43 · f7cde43
1 parent 9d441d0
commit f7cde43
Show file tree

Hide file tree

Showing 8 changed files with 9 additions and 10 deletions.
diff --git a/...client/opensearch/_common/ErrorCause.java → .../client/opensearch/_types/ErrorCause.java b/...client/opensearch/_common/ErrorCause.java → .../client/opensearch/_types/ErrorCause.java
@@ -34,7 +34,7 @@
 // THIS CODE IS GENERATED. MANUAL EDITS WILL BE LOST.
 //----------------------------------------------------
 
-package org.opensearch.client.opensearch._common;
+package org.opensearch.client.opensearch._types;
 
 import jakarta.json.stream.JsonGenerator;
 import java.util.List;

diff --git a/...nt/opensearch/_common/ExpandWildcard.java → ...ent/opensearch/_types/ExpandWildcard.java b/...nt/opensearch/_common/ExpandWildcard.java → ...ent/opensearch/_types/ExpandWildcard.java
@@ -34,7 +34,7 @@
 // THIS CODE IS GENERATED. MANUAL EDITS WILL BE LOST.
 //----------------------------------------------------
 
-package org.opensearch.client.opensearch._common;
+package org.opensearch.client.opensearch._types;
 
 import org.opensearch.client.json.JsonEnum;
 import org.opensearch.client.json.JsonpDeserializable;

diff --git a/...nt/opensearch/_common/NodeStatistics.java → ...ent/opensearch/_types/NodeStatistics.java b/...nt/opensearch/_common/NodeStatistics.java → ...ent/opensearch/_types/NodeStatistics.java
@@ -34,7 +34,7 @@
 // THIS CODE IS GENERATED. MANUAL EDITS WILL BE LOST.
 //----------------------------------------------------
 
-package org.opensearch.client.opensearch._common;
+package org.opensearch.client.opensearch._types;
 
 import jakarta.json.stream.JsonGenerator;
 import java.util.List;

diff --git a/java-client/src/generated/java/org/opensearch/client/opensearch/cat/AliasesRequest.java b/java-client/src/generated/java/org/opensearch/client/opensearch/cat/AliasesRequest.java
@@ -42,8 +42,8 @@
 import java.util.function.Function;
 import java.util.stream.Collectors;
 import javax.annotation.Nullable;
-import org.opensearch.client.opensearch._common.ExpandWildcard;
 import org.opensearch.client.opensearch._types.ErrorResponse;
+import org.opensearch.client.opensearch._types.ExpandWildcard;
 import org.opensearch.client.opensearch._types.RequestBase;
 import org.opensearch.client.transport.Endpoint;
 import org.opensearch.client.transport.endpoints.SimpleEndpoint;

diff --git a/...arch/nodes/_common/NodesResponseBase.java → ...t/opensearch/nodes/NodesResponseBase.java b/...arch/nodes/_common/NodesResponseBase.java → ...t/opensearch/nodes/NodesResponseBase.java
@@ -34,7 +34,7 @@
 // THIS CODE IS GENERATED. MANUAL EDITS WILL BE LOST.
 //----------------------------------------------------
 
-package org.opensearch.client.opensearch.nodes._common;
+package org.opensearch.client.opensearch.nodes;
 
 import jakarta.json.stream.JsonGenerator;
 import java.util.function.Function;
@@ -45,7 +45,7 @@
 import org.opensearch.client.json.JsonpSerializable;
 import org.opensearch.client.json.ObjectBuilderDeserializer;
 import org.opensearch.client.json.ObjectDeserializer;
-import org.opensearch.client.opensearch._common.NodeStatistics;
+import org.opensearch.client.opensearch._types.NodeStatistics;
 import org.opensearch.client.util.ObjectBuilder;
 import org.opensearch.client.util.ObjectBuilderBase;
 

diff --git a/java-client/src/generated/java/org/opensearch/client/opensearch/nodes/UsageResponse.java b/java-client/src/generated/java/org/opensearch/client/opensearch/nodes/UsageResponse.java
@@ -46,7 +46,6 @@
 import org.opensearch.client.json.JsonpSerializable;
 import org.opensearch.client.json.ObjectBuilderDeserializer;
 import org.opensearch.client.json.ObjectDeserializer;
-import org.opensearch.client.opensearch.nodes._common.NodesResponseBase;
 import org.opensearch.client.opensearch.nodes.usage.NodeUsage;
 import org.opensearch.client.util.ApiTypeHelper;
 import org.opensearch.client.util.ObjectBuilder;

diff --git a/java-codegen/src/main/java/org/opensearch/client/codegen/model/Namespace.java b/java-codegen/src/main/java/org/opensearch/client/codegen/model/Namespace.java
@@ -60,8 +60,6 @@ public Namespace child(String name) {
         var childName = idx >= 0 ? name.substring(0, idx) : name;
         var grandChildName = idx >= 0 ? name.substring(idx + 1) : null;
 
-        // if ("_common".equals(childName) && this.parent != null) return child(grandChildName);
-
         Namespace child = children.computeIfAbsent(childName, n -> new Namespace(this, n));
         return grandChildName == null ? child : child.child(grandChildName);
     }

diff --git a/java-codegen/src/main/java/org/opensearch/client/codegen/openapi/OpenApiSchema.java b/java-codegen/src/main/java/org/opensearch/client/codegen/openapi/OpenApiSchema.java
@@ -130,7 +130,9 @@ public String getName() {
 
     public String getNamespace() {
         var schemaFile = Path.of(getParent().getLocation().getPath()).getFileName().toString();
-        return schemaFile.substring(0, schemaFile.lastIndexOf('.'));
+        var ns = schemaFile.substring(0, schemaFile.lastIndexOf('.'));
+        // Match existing naming scheme
+        return ns.replaceAll("\\._common", "").replaceAll("^_common", "_types");
     }
 
     @Override