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