You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Jonathan Ellis (JIRA)" <ji...@apache.org> on 2009/12/17 03:28:18 UTC
[jira] Commented: (CASSANDRA-638) Check SlicePredicate/ColumnParent
Column versus SuperColumn consistency
[ https://issues.apache.org/jira/browse/CASSANDRA-638?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12791744#action_12791744 ]
Jonathan Ellis commented on CASSANDRA-638:
------------------------------------------
Do you think you could add a test to test/system/test_server.py illustrating what should be rejected?
(http://wiki.apache.org/cassandra/HowToContribute explains how to run the system tests)
> Check SlicePredicate/ColumnParent Column versus SuperColumn consistency
> ------------------------------------------------------------------------
>
> Key: CASSANDRA-638
> URL: https://issues.apache.org/jira/browse/CASSANDRA-638
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Adam Fisk
> Priority: Minor
> Fix For: 0.9
>
>
> This crops up in any call taking both a ColumnParent and a SlicePredicate, as some settings apply only to Columns while others apply only to SuperColumns.
> For example, it doesn't make sense to call setColumn_names on the SlicePredicate when the ColumnParent is a ColumnFamily that contains SuperColumns, as the return values will be SuperColumns, not Columns. The resulting error is currently difficult to decipher (16 byte UUID required).
> Similarly, I'm not sure what happens if you don't call setColumn_names on the SlicePredicate in the case where you are querying a ColumnFamily with Columns, although I'm guessing it returns all column names.
> Just a quick check to make sure the SlicePredicate doesn't have column names set when the ColumnParent is a SuperColumnFamily, returning a more informative error if so, should do the trick.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.