Mailing-List: contact commits-help@cassandra.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cassandra.apache.org
Date: Wed, 28 Oct 2015 15:40:27 +0000 (UTC)
From: "Sylvain Lebresne (JIRA)" <jira@apache.org>
To: commits@cassandra.apache.org
Message-ID: <JIRA.12907411.1445609150000.85088.1446046827715@Atlassian.JIRA>
In-Reply-To: <JIRA.12907411.1445609150000@Atlassian.JIRA>
References: <JIRA.12907411.1445609150000@Atlassian.JIRA>
 <JIRA.12907411.1445609150896@arcas>
Subject: [jira] [Commented] (CASSANDRA-10576) Thrift CAS on static columns
 doesn't work as expected
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

Sylvain Lebresne commented on CASSANDRA-10576:
----------------------------------------------

+1

Nit: in {{ThriftCasRequest}}, there is 2 places where we do {{if (expected == null || expected.isEmpty())}}: we can ditch the {{expected == null}} now. 

> Thrift CAS on static columns doesn't work as expected
> -----------------------------------------------------
>
>                 Key: CASSANDRA-10576
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-10576
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Sam Tunnicliffe
>            Assignee: Sam Tunnicliffe
>             Fix For: 3.0.0
>
>
> Although the thrift cas call works as expected for dynamic column families, using it on tables with statically defined columns produces unexpected results. The problem, in {{ThriftCASRequest}}, is that while the {{expected}} partition contains a static row, the {{current}} partition has been processed by {{ThriftResultsMerger}} during the read, converting the static columns to clusterings. If {{expected}} contains only a static row, no further checking is carried out, {{appliesTo}} erroneously returns true and the conditional update is performed regardless of the current state of the partition.


--
This message was sent by Atlassian JIRA
(v6.3.4#6332)