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: 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

Reply via email to