smiklosovic commented on code in PR #4115:
URL: https://github.com/apache/cassandra/pull/4115#discussion_r2073121241


##########
src/java/org/apache/cassandra/cql3/constraints/ConstraintFunction.java:
##########
@@ -100,4 +113,44 @@ public void validate(ColumnMetadata columnMetadata, String 
term) throws InvalidC
      * @return supported types for given constraint
      */
     public abstract List<AbstractType<?>> getSupportedTypes();
+
+    /**
+     * Tells whether implementation supports specifying arguments on its 
function.
+     * <br>
+     * In this case, this function will return "true"
+     * <pre>
+     *     val int check length() < 1024
+     * </pre>
+     *
+     * In this case, this function will return "false"
+     * <pre>
+     *     val int check someconstraint('abc', 'def')
+     * </pre>
+     * @return true if this constraint does not accept any parameters, false 
otherwise.
+     */
+    public boolean isParameterless() { return true; }

Review Comment:
   That is possible yes but I do not think it is a big deal to just extend and 
override for now. 



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