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

Aleksey Yeschenko commented on CASSANDRA-5300:
----------------------------------------------

bq. I'd prefer doing the validation in Operation.SetValue directly because that 
where we have all the other type validation now (and it's cleaner that way 
imo), as done in the attached v2.

wfm, +1
                
> Insufficient validation of UPDATE queries against counter cfs
> -------------------------------------------------------------
>
>                 Key: CASSANDRA-5300
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-5300
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: 1.2.0
>            Reporter: Aleksey Yeschenko
>            Assignee: Aleksey Yeschenko
>             Fix For: 1.2.3
>
>         Attachments: 5300.txt, 5300-v2.txt
>
>
> {noformat}
> CREATE TABLE demo (
>   id int PRIMARY KEY,
>   c counter
> )
> {noformat}
> This is expected:
> {noformat}
> insert into demo (id, c) VALUES ( 0, 20);
> Bad Request: INSERT statement are not allowed on counter tables, use UPDATE 
> instead
> {noformat}
> This should also be forbidden, but it is not:
> {noformat}
> update demo set c = 20 where id = 0;
> select * from demo;
>  id | c
> ----+----
>   0 | 20
> {noformat}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to