bbotella commented on code in PR #3921:
URL: https://github.com/apache/cassandra/pull/3921#discussion_r1972032420
##########
src/java/org/apache/cassandra/cql3/statements/schema/AlterTableStatement.java:
##########
@@ -739,9 +739,9 @@ public KeyspaceMetadata apply(Epoch epoch, KeyspaceMetadata
keyspace, TableMetad
public static class AlterConstraints extends AlterTableStatement
{
final ColumnIdentifier columnName;
- final ColumnConstraints constraints;
+ final ColumnConstraints.Raw constraints;
- AlterConstraints(String keyspaceName, String tableName, boolean
ifTableExists, ColumnIdentifier columnName, ColumnConstraints constraints)
+ AlterConstraints(String keyspaceName, String tableName, boolean
ifTableExists, ColumnIdentifier columnName, ColumnConstraints.Raw constraints)
Review Comment:
Added a test and it works:
```
@Test
public void testCreateTableAddConstraintWithIfExists() throws Throwable
{
createTable("CREATE TABLE %s (pk int, ck1 int, ck2 int, v int,
PRIMARY KEY ((pk),ck1, ck2)) WITH CLUSTERING ORDER BY (ck1 ASC);");
execute("ALTER TABLE %s ALTER IF EXISTS foo CHECK foo < 100");
}
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org
For additional commands, e-mail: pr-h...@cassandra.apache.org
