Apply suggestions

Author: KodaiD

core/src/main/java/com/scalar/db/storage/jdbc/JdbcAdmin.java

@@ -939,7 +939,7 @@ public void renameTable(String namespace, String oldTableName, String newTableNa
       String renameTableStatement = rdbEngine.renameTableSql(namespace, oldTableName, newTableName);
       try (Connection connection = dataSource.getConnection()) {
         execute(connection, renameTableStatement);
-        execute(connection, getDeleteTableMetadataStatement(namespace, oldTableName));
+        deleteTableMetadata(connection, namespace, oldTableName);
         for (String indexedColumnName : tableMetadata.getSecondaryIndexNames()) {
           String oldIndexName = getIndexName(namespace, oldTableName, indexedColumnName);
           String newIndexName = getIndexName(namespace, newTableName, indexedColumnName);

core/src/main/java/com/scalar/db/transaction/consensuscommit/ConsensusCommitAdmin.java

@@ -282,12 +282,6 @@ public void renameTable(String namespace, String oldTableName, String newTableNa
       throws ExecutionException {
     checkNamespace(namespace);
 
-    TableMetadata tableMetadata = getTableMetadata(namespace, oldTableName);
-    if (tableMetadata == null) {
-      throw new IllegalArgumentException(
-          CoreError.TABLE_NOT_FOUND.buildMessage(
-              ScalarDbUtils.getFullTableName(namespace, oldTableName)));
-    }
     admin.renameTable(namespace, oldTableName, newTableName);
   }
 

integration-test/src/main/java/com/scalar/db/api/DistributedStorageAdminIntegrationTestBase.java

@@ -1383,6 +1383,8 @@ public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesC
       assertThat(admin.tableExists(namespace1, getTable4())).isFalse();
       assertThat(admin.tableExists(namespace1, newTableName)).isTrue();
       assertThat(admin.getTableMetadata(namespace1, newTableName)).isEqualTo(tableMetadata);
+      assertThat(admin.indexExists(namespace1, getTable4(), getColumnName2())).isFalse();
+      assertThat(admin.indexExists(namespace1, getTable4(), getColumnName3())).isFalse();
       assertThat(admin.indexExists(namespace1, newTableName, getColumnName2())).isTrue();
       assertThat(admin.indexExists(namespace1, newTableName, getColumnName3())).isTrue();
       assertThatCode(() -> admin.dropIndex(namespace1, newTableName, getColumnName2()))

integration-test/src/main/java/com/scalar/db/api/DistributedStorageAdminPermissionIntegrationTestBase.java

@@ -353,7 +353,7 @@ public void renameTable_WithSufficientPermission_ShouldSucceed() throws Executio
     // Arrange
     createNamespaceByRoot();
     createTableByRoot();
-    String newTableName = TABLE + "_new";
+    String newTableName = "new_" + TABLE;
 
     // Act Assert
     assertThatCode(() -> adminForNormalUser.renameTable(NAMESPACE, TABLE, newTableName))