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

Jonathan Ellis commented on CASSANDRA-3036:
-------------------------------------------

- let's name the ColumnDef variable in validateCfDef something other than 
"column" (which usually means an IColumn or similar)
- when we throw an invalid column name error, we should use the CF's comparator 
rather than AsciiType (which will error out on many binary column names)
- CCFS calls getByteBuffer which catches MarshallException and turns it into 
IRE.  Suggest doing the validation ones from CCFS.validate, that way we don't 
have to do it again in CCFS.getColumns.  (Can just call AbstractType.fromString 
directly)
- go ahead and commit the println removal separately :)

> Vague primary key references in CQL
> -----------------------------------
>
>                 Key: CASSANDRA-3036
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3036
>             Project: Cassandra
>          Issue Type: Bug
>          Components: API
>    Affects Versions: 0.8.1
>            Reporter: Kelley Reynolds
>            Assignee: Pavel Yaskevich
>            Priority: Minor
>              Labels: core, cql
>             Fix For: 0.8.5
>
>         Attachments: CASSANDRA-3036.patch
>
>
> create columnfamily wonk (id 'utf8' primary key, id int)
> update wonk set id=1 where id='test'
> create index wonk_id on wonk (id)
> This does what you would expect but then the results are unclear when using 
> 'id' in a where clause.
> "select * from wonk where id=1" returns nothing and "select * from wonk where 
> id='test'" works fine.
> Perhaps secondary indexes should not be allowed on columns that have the same 
> name as the key_alias? At least a warning or something should be thrown to 
> indicate you've just made a useless index.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to