smiklosovic commented on code in PR #4115:
URL: https://github.com/apache/cassandra/pull/4115#discussion_r2058620968
##########
src/java/org/apache/cassandra/cql3/constraints/ColumnConstraints.java:
##########
@@ -216,20 +224,14 @@ public ColumnConstraints prepare(ColumnIdentifier column)
if (constraints.isEmpty())
return NO_OP;
- for (ColumnConstraint<?> constraint : constraints)
- {
- if (constraint.columnName != null &&
!column.equals(constraint.columnName))
- throw new
InvalidConstraintDefinitionException(format("Constraint %s was not specified on
a column it operates on: %s but on: %s",
-
constraint, column.toCQLString(), constraint.columnName));
- }
-
- return new ColumnConstraints(constraints);
+ ColumnConstraints columnConstraints = new
ColumnConstraints(constraints);
Review Comment:
the main magic here, we defer column binding here.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
