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

Jonathan Ellis commented on CASSANDRA-3761:
-------------------------------------------

bq. I'll note that the problem is the same for indexed queries

Right, "pagination" client-side is broken there as well.  Again, I think we 
should disallow it for now and add it in when we have a solution we're happy 
with.  (Maybe that even means the existing syntax, but I'm not convinced of 
that.)

bq. internally a dynamic CF don't use a composite, so it is bound to have some 
differences internally

Maybe.  In my mind I'd rather make that distinction at the "Okay we have 
Kind.DENSE, now do I have to treat this code differently based on cf.comparator 
instanceof compositetype?"  But I'm happy to see what it looks like with your 
Builder approach.

(That reminds me -- we should probably go with COMPACT instead of DENSE, to 
match the CQL syntax.)

bq. you will only get one record back, the one where col1 is 'c' and col2 is 
not defined, if it exists, not all the record that have as prefix col1 == 'c'

That sounds like a bug to me.  If I wanted to restrict it to col2 not defined, 
I should have to write {{SELECT * FROM foo WHERE key = 'k' AND col1 = 'c' AND 
col2 = null}}.  Which I am fine with leaving out for now, since we've ignored 
{{null}} so far in CQL, but the behavior of {{SELECT * FROM foo WHERE key = 'k' 
AND col1 = 'c'}} should definitely be "all columns with prefix c."  (So maybe 
we need to disallow that too temporarily, until we've done some of the work for 
CASSANDRA-3237.)
                
> CQL 3.0
> -------
>
>                 Key: CASSANDRA-3761
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3761
>             Project: Cassandra
>          Issue Type: New Feature
>          Components: API
>            Reporter: Sylvain Lebresne
>              Labels: cql
>             Fix For: 1.1
>
>         Attachments: 0001-CQL-3.0.patch, 
> 0002-Add-support-for-switching-the-CQL-version.patch, 
> 0003-Makes-batches-atomic.patch, 0004-Thrift-gen-files.patch, cql_tests.py, 
> create_cf_syntaxes.txt
>
>
> This ticket is a reformulation/generalization of CASSANDRA-2474. The core 
> change of CQL 3.0 is to introduce the new syntaxes that were discussed in 
> CASSANDRA-2474 that allow to:
> # Provide a better/more native support for wide rows, using the idea of 
> transposed vie.
> # The generalization to composite columns.
> The attached text file create_cf_syntaxes.txt recall the new syntaxes 
> introduced.
> The changes proposed above allow (and strongly suggest in some cases) a 
> number of other changes to the language that this ticket proposes to 
> explore/implement (more details coming in the comments).

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to