You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Sylvain Lebresne (JIRA)" <ji...@apache.org> on 2014/02/04 18:46:12 UTC

[jira] [Created] (CASSANDRA-6649) CQL: disallow counter update with "USING TIMESTAMP" and "USING TTL"

Sylvain Lebresne created CASSANDRA-6649:
-------------------------------------------

             Summary: CQL: disallow counter update with "USING TIMESTAMP" and "USING TTL"
                 Key: CASSANDRA-6649
                 URL: https://issues.apache.org/jira/browse/CASSANDRA-6649
             Project: Cassandra
          Issue Type: Bug
            Reporter: Sylvain Lebresne
            Priority: Minor
             Fix For: 1.2.15, 2.0.6


Timestamps are not used by counters and TTL are not supported, but it appears we don't reject counter updates that have "USING TIMESTAMP X" or "USING TTL X". We should since both are non-sensical (the value is completely ignored currently).

Note: we should also refuse "USING TIMESTAMP" on "DELETE" statements on counters table: even though we kind of do use a timestamp internally, it's more of an implementation detail and in fact may go away with CASSANDRA-6506 (there is also nothing clever you can do with it by providing it client side).

Note bis: strictly speaking doing that could break a few users that where setting those thinking it does something. I think that the lack of validation is more of a bug and that user that think it's doing something probably ought to know it's not sooner than later, but I could be fine with just warning in the log file for 1.2 and 2.0, and only rejecting in 2.1 if someone thinks it's safer.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)