[rest] Remove useless CreateDatabaseResponse (#5318)

Author: JingsongLi

paimon-core/src/main/java/org/apache/paimon/rest/RESTCatalog.java

@@ -55,7 +55,6 @@
 import org.apache.paimon.rest.responses.AlterDatabaseResponse;
 import org.apache.paimon.rest.responses.CommitTableResponse;
 import org.apache.paimon.rest.responses.ConfigResponse;
-import org.apache.paimon.rest.responses.CreateDatabaseResponse;
 import org.apache.paimon.rest.responses.ErrorResponseResourceType;
 import org.apache.paimon.rest.responses.GetDatabaseResponse;
 import org.apache.paimon.rest.responses.GetTableResponse;
@@ -207,11 +206,7 @@ public void createDatabase(String name, boolean ignoreIfExists, Map<String, Stri
         checkNotSystemDatabase(name);
         CreateDatabaseRequest request = new CreateDatabaseRequest(name, properties);
         try {
-            client.post(
-                    resourcePaths.databases(),
-                    request,
-                    CreateDatabaseResponse.class,
-                    restAuthFunction);
+            client.post(resourcePaths.databases(), request, restAuthFunction);
         } catch (AlreadyExistsException e) {
             if (!ignoreIfExists) {
                 throw new DatabaseAlreadyExistException(name);

paimon-core/src/test/java/org/apache/paimon/rest/MockRESTMessage.java

@@ -27,7 +27,6 @@
 import org.apache.paimon.rest.requests.CreateViewRequest;
 import org.apache.paimon.rest.requests.RenameTableRequest;
 import org.apache.paimon.rest.responses.AlterDatabaseResponse;
-import org.apache.paimon.rest.responses.CreateDatabaseResponse;
 import org.apache.paimon.rest.responses.GetDatabaseResponse;
 import org.apache.paimon.rest.responses.GetTableResponse;
 import org.apache.paimon.rest.responses.GetTableTokenResponse;
@@ -72,12 +71,6 @@ public static CreateDatabaseRequest createDatabaseRequest(String name) {
         return new CreateDatabaseRequest(name, options);
     }
 
-    public static CreateDatabaseResponse createDatabaseResponse(String name) {
-        Map<String, String> options = new HashMap<>();
-        options.put("a", "b");
-        return new CreateDatabaseResponse(name, options);
-    }
-
     public static GetDatabaseResponse getDatabaseResponse(String name) {
         Map<String, String> options = new HashMap<>();
         options.put("a", "b");

paimon-core/src/test/java/org/apache/paimon/rest/RESTCatalogServer.java

@@ -52,7 +52,6 @@
 import org.apache.paimon.rest.responses.AlterDatabaseResponse;
 import org.apache.paimon.rest.responses.CommitTableResponse;
 import org.apache.paimon.rest.responses.ConfigResponse;
-import org.apache.paimon.rest.responses.CreateDatabaseResponse;
 import org.apache.paimon.rest.responses.ErrorResponse;
 import org.apache.paimon.rest.responses.ErrorResponseResourceType;
 import org.apache.paimon.rest.responses.GetDatabaseResponse;
@@ -571,7 +570,6 @@ private MockResponse commitTableHandle(Identifier identifier, String data) throw
 
     private MockResponse databasesApiHandler(
             String method, String data, Map<String, String> parameters) throws Exception {
-        RESTResponse response;
         switch (method) {
             case "GET":
                 List<String> databases = new ArrayList<>(databaseStore.keySet());
@@ -586,8 +584,7 @@ private MockResponse databasesApiHandler(
                 catalog.createDatabase(databaseName, false);
                 databaseStore.put(
                         databaseName, Database.of(databaseName, requestBody.getOptions(), null));
-                response = new CreateDatabaseResponse(databaseName, requestBody.getOptions());
-                return mockResponse(response, 200);
+                return new MockResponse().setResponseCode(200);
             default:
                 return new MockResponse().setResponseCode(404);
         }

paimon-core/src/test/java/org/apache/paimon/rest/RESTObjectMapperTest.java

@@ -26,7 +26,6 @@
 import org.apache.paimon.rest.requests.RenameTableRequest;
 import org.apache.paimon.rest.responses.AlterDatabaseResponse;
 import org.apache.paimon.rest.responses.ConfigResponse;
-import org.apache.paimon.rest.responses.CreateDatabaseResponse;
 import org.apache.paimon.rest.responses.ErrorResponse;
 import org.apache.paimon.rest.responses.GetDatabaseResponse;
 import org.apache.paimon.rest.responses.GetTableResponse;
@@ -84,17 +83,6 @@ public void createDatabaseRequestParseTest() throws Exception {
         assertEquals(request.getOptions().size(), parseData.getOptions().size());
     }
 
-    @Test
-    public void createDatabaseResponseParseTest() throws Exception {
-        String name = MockRESTMessage.databaseName();
-        CreateDatabaseResponse response = MockRESTMessage.createDatabaseResponse(name);
-        String responseStr = OBJECT_MAPPER.writeValueAsString(response);
-        CreateDatabaseResponse parseData =
-                OBJECT_MAPPER.readValue(responseStr, CreateDatabaseResponse.class);
-        assertEquals(name, parseData.getName());
-        assertEquals(response.getOptions().size(), parseData.getOptions().size());
-    }
-
     @Test
     public void getDatabaseResponseParseTest() throws Exception {
         String name = MockRESTMessage.databaseName();

paimon-open-api/rest-catalog-open-api.yaml

@@ -234,11 +234,7 @@ paths:
               $ref: '#/components/schemas/CreateDatabaseRequest'
       responses:
         "200":
-          description: OK
-          content:
-            application/json:
-              schema:
-                $ref: '#/components/schemas/CreateDatabaseResponse'
+          description: Success, no content
         "409":
           description: Resource has exist
           content:
@@ -1430,8 +1426,6 @@ components:
           format: int64
         schema:
           $ref: '#/components/schemas/Schema'
-        uuid:
-          type: string
         owner:
           type: string
         createdAt:

paimon-open-api/src/main/java/org/apache/paimon/open/api/RESTCatalogController.java

@@ -32,7 +32,6 @@
 import org.apache.paimon.rest.responses.AlterDatabaseResponse;
 import org.apache.paimon.rest.responses.CommitTableResponse;
 import org.apache.paimon.rest.responses.ConfigResponse;
-import org.apache.paimon.rest.responses.CreateDatabaseResponse;
 import org.apache.paimon.rest.responses.ErrorResponse;
 import org.apache.paimon.rest.responses.GetDatabaseResponse;
 import org.apache.paimon.rest.responses.GetTableResponse;
@@ -125,11 +124,7 @@ public ListDatabasesResponse listDatabases(
             summary = "Create Databases",
             tags = {"database"})
     @ApiResponses({
-        @ApiResponse(
-                responseCode = "200",
-                content = {
-                    @Content(schema = @Schema(implementation = CreateDatabaseResponse.class))
-                }),
+        @ApiResponse(responseCode = "200", description = "Success, no content"),
         @ApiResponse(
                 responseCode = "409",
                 description = "Resource has exist",
@@ -139,11 +134,8 @@ public ListDatabasesResponse listDatabases(
                 content = {@Content(schema = @Schema())})
     })
     @PostMapping("/v1/{prefix}/databases")
-    public CreateDatabaseResponse createDatabases(
-            @PathVariable String prefix, @RequestBody CreateDatabaseRequest request) {
-        Map<String, String> properties = new HashMap<>();
-        return new CreateDatabaseResponse("name", properties);
-    }
+    public void createDatabases(
+            @PathVariable String prefix, @RequestBody CreateDatabaseRequest request) {}
 
     @Operation(
             summary = "Get Database",