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

["Aleksey Yeschenko (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/CASSANDRA-6649?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13890982#comment-13890982 ] 

Aleksey Yeschenko commented on CASSANDRA-6649:
----------------------------------------------

Probably should leave 1.2 alone. Maybe 2.0 as well, since some might be using it (foolishly), and introduce the change as part of the 'counters 2.0'.

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