Skip to content

Commit bb3de17

Browse files
authored
Add rename table (#3021)
1 parent f1765d6 commit bb3de17

File tree

45 files changed

+893
-45
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

45 files changed

+893
-45
lines changed

core/src/integration-test/java/com/scalar/db/storage/cassandra/CassandraAdminCaseSensitivityIntegrationTest.java

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -76,4 +76,20 @@ public void renameColumn_ForIndexKeyColumn_ShouldRenameColumnAndIndexCorrectly()
7676
admin.dropTable(getNamespace1(), getTable4(), true);
7777
}
7878
}
79+
80+
@Override
81+
@Disabled("Cassandra does not support renaming tables")
82+
public void renameTable_ForExistingTable_ShouldRenameTableCorrectly() {}
83+
84+
@Override
85+
@Disabled("Cassandra does not support renaming tables")
86+
public void renameTable_ForNonExistingTable_ShouldThrowIllegalArgumentException() {}
87+
88+
@Override
89+
@Disabled("Cassandra does not support renaming tables")
90+
public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentException() {}
91+
92+
@Override
93+
@Disabled("Cassandra does not support renaming tables")
94+
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
7995
}

core/src/integration-test/java/com/scalar/db/storage/cassandra/CassandraAdminIntegrationTest.java

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,4 +75,20 @@ public void renameColumn_ForIndexKeyColumn_ShouldRenameColumnAndIndexCorrectly()
7575
admin.dropTable(getNamespace1(), getTable4(), true);
7676
}
7777
}
78+
79+
@Override
80+
@Disabled("Cassandra does not support renaming tables")
81+
public void renameTable_ForExistingTable_ShouldRenameTableCorrectly() {}
82+
83+
@Override
84+
@Disabled("Cassandra does not support renaming tables")
85+
public void renameTable_ForNonExistingTable_ShouldThrowIllegalArgumentException() {}
86+
87+
@Override
88+
@Disabled("Cassandra does not support renaming tables")
89+
public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentException() {}
90+
91+
@Override
92+
@Disabled("Cassandra does not support renaming tables")
93+
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
7894
}

core/src/integration-test/java/com/scalar/db/storage/cassandra/CassandraAdminPermissionIntegrationTest.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -153,4 +153,9 @@ public void renameColumn_WithSufficientPermission_ShouldSucceed() throws Executi
153153
assertThatCode(() -> adminForNormalUser.renameColumn(NAMESPACE, TABLE, COL_NAME1, NEW_COL_NAME))
154154
.doesNotThrowAnyException();
155155
}
156+
157+
@Test
158+
@Override
159+
@Disabled("Cassandra does not support renaming tables")
160+
public void renameTable_WithSufficientPermission_ShouldSucceed() {}
156161
}

core/src/integration-test/java/com/scalar/db/storage/cassandra/ConsensusCommitAdminIntegrationTestWithCassandra.java

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,4 +75,20 @@ public void renameColumn_ForIndexKeyColumn_ShouldRenameColumnAndIndexCorrectly()
7575
admin.dropTable(namespace1, TABLE4, true);
7676
}
7777
}
78+
79+
@Override
80+
@Disabled("Cassandra does not support renaming tables")
81+
public void renameTable_ForExistingTable_ShouldRenameTableCorrectly() {}
82+
83+
@Override
84+
@Disabled("Cassandra does not support renaming tables")
85+
public void renameTable_ForNonExistingTable_ShouldThrowIllegalArgumentException() {}
86+
87+
@Override
88+
@Disabled("Cassandra does not support renaming tables")
89+
public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentException() {}
90+
91+
@Override
92+
@Disabled("Cassandra does not support renaming tables")
93+
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
7894
}

core/src/integration-test/java/com/scalar/db/storage/cassandra/SingleCrudOperationTransactionAdminIntegrationTestWithCassandra.java

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -70,4 +70,20 @@ public void renameColumn_ForIndexKeyColumn_ShouldRenameColumnAndIndexCorrectly()
7070
admin.dropTable(namespace1, TABLE4, true);
7171
}
7272
}
73+
74+
@Override
75+
@Disabled("Cassandra does not support renaming tables")
76+
public void renameTable_ForExistingTable_ShouldRenameTableCorrectly() {}
77+
78+
@Override
79+
@Disabled("Cassandra does not support renaming tables")
80+
public void renameTable_ForNonExistingTable_ShouldThrowIllegalArgumentException() {}
81+
82+
@Override
83+
@Disabled("Cassandra does not support renaming tables")
84+
public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentException() {}
85+
86+
@Override
87+
@Disabled("Cassandra does not support renaming tables")
88+
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
7389
}

core/src/integration-test/java/com/scalar/db/storage/cosmos/ConsensusCommitAdminIntegrationTestWithCosmos.java

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -67,4 +67,20 @@ public void renameColumn_ForPrimaryKeyColumn_ShouldRenameColumnCorrectly() {}
6767
@Override
6868
@Disabled("Cosmos DB does not support renaming columns")
6969
public void renameColumn_ForIndexKeyColumn_ShouldRenameColumnAndIndexCorrectly() {}
70+
71+
@Override
72+
@Disabled("Cosmos DB does not support renaming tables")
73+
public void renameTable_ForExistingTable_ShouldRenameTableCorrectly() {}
74+
75+
@Override
76+
@Disabled("Cosmos DB does not support renaming tables")
77+
public void renameTable_ForNonExistingTable_ShouldThrowIllegalArgumentException() {}
78+
79+
@Override
80+
@Disabled("Cosmos DB does not support renaming tables")
81+
public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentException() {}
82+
83+
@Override
84+
@Disabled("Cosmos DB does not support renaming tables")
85+
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
7086
}

core/src/integration-test/java/com/scalar/db/storage/cosmos/CosmosAdminCaseSensitivityIntegrationTest.java

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -67,4 +67,20 @@ public void renameColumn_ForPrimaryKeyColumn_ShouldRenameColumnCorrectly() {}
6767
@Override
6868
@Disabled("Cosmos DB does not support renaming columns")
6969
public void renameColumn_ForIndexKeyColumn_ShouldRenameColumnAndIndexCorrectly() {}
70+
71+
@Override
72+
@Disabled("Cosmos DB does not support renaming tables")
73+
public void renameTable_ForExistingTable_ShouldRenameTableCorrectly() {}
74+
75+
@Override
76+
@Disabled("Cosmos DB does not support renaming tables")
77+
public void renameTable_ForNonExistingTable_ShouldThrowIllegalArgumentException() {}
78+
79+
@Override
80+
@Disabled("Cosmos DB does not support renaming tables")
81+
public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentException() {}
82+
83+
@Override
84+
@Disabled("Cosmos DB does not support renaming tables")
85+
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
7086
}

core/src/integration-test/java/com/scalar/db/storage/cosmos/CosmosAdminIntegrationTest.java

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -66,4 +66,20 @@ public void renameColumn_ForPrimaryKeyColumn_ShouldRenameColumnCorrectly() {}
6666
@Override
6767
@Disabled("Cosmos DB does not support renaming columns")
6868
public void renameColumn_ForIndexKeyColumn_ShouldRenameColumnAndIndexCorrectly() {}
69+
70+
@Override
71+
@Disabled("Cosmos DB does not support renaming tables")
72+
public void renameTable_ForExistingTable_ShouldRenameTableCorrectly() {}
73+
74+
@Override
75+
@Disabled("Cosmos DB does not support renaming tables")
76+
public void renameTable_ForNonExistingTable_ShouldThrowIllegalArgumentException() {}
77+
78+
@Override
79+
@Disabled("Cosmos DB does not support renaming tables")
80+
public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentException() {}
81+
82+
@Override
83+
@Disabled("Cosmos DB does not support renaming tables")
84+
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
6985
}

core/src/integration-test/java/com/scalar/db/storage/cosmos/SingleCrudOperationTransactionAdminIntegrationTestWithCosmos.java

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,4 +61,20 @@ public void renameColumn_ForPrimaryKeyColumn_ShouldRenameColumnCorrectly() {}
6161
@Override
6262
@Disabled("Cosmos DB does not support renaming columns")
6363
public void renameColumn_ForIndexKeyColumn_ShouldRenameColumnAndIndexCorrectly() {}
64+
65+
@Override
66+
@Disabled("Cosmos DB does not support renaming tables")
67+
public void renameTable_ForExistingTable_ShouldRenameTableCorrectly() {}
68+
69+
@Override
70+
@Disabled("Cosmos DB does not support renaming tables")
71+
public void renameTable_ForNonExistingTable_ShouldThrowIllegalArgumentException() {}
72+
73+
@Override
74+
@Disabled("Cosmos DB does not support renaming tables")
75+
public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentException() {}
76+
77+
@Override
78+
@Disabled("Cosmos DB does not support renaming tables")
79+
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
6480
}

core/src/integration-test/java/com/scalar/db/storage/dynamo/ConsensusCommitAdminIntegrationTestWithDynamo.java

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -72,4 +72,20 @@ public void renameColumn_ForPrimaryKeyColumn_ShouldRenameColumnCorrectly() {}
7272
@Override
7373
@Disabled("DynamoDB does not support renaming columns")
7474
public void renameColumn_ForIndexKeyColumn_ShouldRenameColumnAndIndexCorrectly() {}
75+
76+
@Override
77+
@Disabled("DynamoDB does not support renaming tables")
78+
public void renameTable_ForExistingTable_ShouldRenameTableCorrectly() {}
79+
80+
@Override
81+
@Disabled("DynamoDB does not support renaming tables")
82+
public void renameTable_ForNonExistingTable_ShouldThrowIllegalArgumentException() {}
83+
84+
@Override
85+
@Disabled("DynamoDB does not support renaming tables")
86+
public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentException() {}
87+
88+
@Override
89+
@Disabled("DynamoDB does not support renaming tables")
90+
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
7591
}

0 commit comments

Comments
 (0)