Skip to content

Commit 200f28b

Browse files
feeblefakieKodaiD
andauthored
Backport to branch(3) : Fix to not drop the metadata table when renaming table (#3078)
Co-authored-by: Kodai Doki <[email protected]>
1 parent caddaf4 commit 200f28b

16 files changed

+115
-10
lines changed

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

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -101,4 +101,8 @@ public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentEx
101101
@Override
102102
@Disabled("Cassandra does not support renaming tables")
103103
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
104+
105+
@Override
106+
@Disabled("Cassandra does not support renaming tables")
107+
public void renameTable_IfOnlyOneTableExists_ShouldRenameTableCorrectly() {}
104108
}

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

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -100,4 +100,8 @@ public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentEx
100100
@Override
101101
@Disabled("Cassandra does not support renaming tables")
102102
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
103+
104+
@Override
105+
@Disabled("Cassandra does not support renaming tables")
106+
public void renameTable_IfOnlyOneTableExists_ShouldRenameTableCorrectly() {}
103107
}

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

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -118,4 +118,8 @@ public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentEx
118118
@Override
119119
@Disabled("Cassandra does not support renaming tables")
120120
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
121+
122+
@Override
123+
@Disabled("Cassandra does not support renaming tables")
124+
public void renameTable_IfOnlyOneTableExists_ShouldRenameTableCorrectly() {}
121125
}

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

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -105,4 +105,8 @@ public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentEx
105105
@Override
106106
@Disabled("Cassandra does not support renaming tables")
107107
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
108+
109+
@Override
110+
@Disabled("Cassandra does not support renaming tables")
111+
public void renameTable_IfOnlyOneTableExists_ShouldRenameTableCorrectly() {}
108112
}

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

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -113,4 +113,8 @@ public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentEx
113113
@Override
114114
@Disabled("Cosmos DB does not support renaming tables")
115115
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
116+
117+
@Override
118+
@Disabled("Cosmos DB does not support renaming tables")
119+
public void renameTable_IfOnlyOneTableExists_ShouldRenameTableCorrectly() {}
116120
}

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

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -98,4 +98,8 @@ public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentEx
9898
@Override
9999
@Disabled("Cosmos DB does not support renaming tables")
100100
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
101+
102+
@Override
103+
@Disabled("Cosmos DB does not support renaming tables")
104+
public void renameTable_IfOnlyOneTableExists_ShouldRenameTableCorrectly() {}
101105
}

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

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -97,4 +97,8 @@ public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentEx
9797
@Override
9898
@Disabled("Cosmos DB does not support renaming tables")
9999
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
100+
101+
@Override
102+
@Disabled("Cosmos DB does not support renaming tables")
103+
public void renameTable_IfOnlyOneTableExists_ShouldRenameTableCorrectly() {}
100104
}

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

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -98,4 +98,8 @@ public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentEx
9898
@Override
9999
@Disabled("Cosmos DB does not support renaming tables")
100100
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
101+
102+
@Override
103+
@Disabled("Cosmos DB does not support renaming tables")
104+
public void renameTable_IfOnlyOneTableExists_ShouldRenameTableCorrectly() {}
101105
}

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

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -167,4 +167,8 @@ public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentEx
167167
@Override
168168
@Disabled("DynamoDB does not support renaming tables")
169169
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
170+
171+
@Override
172+
@Disabled("DynamoDB does not support renaming tables")
173+
public void renameTable_IfOnlyOneTableExists_ShouldRenameTableCorrectly() {}
170174
}

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

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -152,4 +152,8 @@ public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentEx
152152
@Override
153153
@Disabled("DynamoDB does not support renaming tables")
154154
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
155+
156+
@Override
157+
@Disabled("DynamoDB does not support renaming tables")
158+
public void renameTable_IfOnlyOneTableExists_ShouldRenameTableCorrectly() {}
155159
}

0 commit comments

Comments
 (0)