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

Eric Evans commented on CASSANDRA-3761:
---------------------------------------

I'm still looking over this, but I thought I'd mention that I'm unable to get 
the tests to pass (most of them produce errors).   For example:

{noformat}
$ nosetests -sxv cql_tests.py 
cql_tests.TestCQL.counters_test ... ERROR

======================================================================
ERROR: cql_tests.TestCQL.counters_test
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.7/nose/case.py", line 187, in runTest
    self.test(*self.arg)
  File "/home/eevans/dev/src/git/cassandra/cql_tests.py", line 339, in 
counters_test
    cursor.execute("UPDATE clicks SET total = total + 1 WHERE userid = 1 AND 
url = 'http://foo.com'")
  File "/usr/local/lib/python2.7/dist-packages/cql/cursor.py", line 89, in 
execute
    raise cql.ProgrammingError("Bad Request: %s" % ire.why)
ProgrammingError: Bad Request: line 1:79 mismatched character '<EOF>' expecting 
'''

----------------------------------------------------------------------
Ran 1 test in 1.871s

FAILED (errors=1)
{noformat}

In this case, the error actually seems to make sense.  Since {{//}} is 
recognized as a comment.  Did the wrong version of the tests get attached maybe?

Another example is:

{noformat}
NameError: global name 'assert_invalid' is not defined
{noformat}

Where is {{assert_invalid}} meant to come from?

Also, I think the patches needs to be rebased in the wake of #3689 (although I 
think the conflicts are minor).

Thanks!
                
> 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
>            Assignee: Sylvain Lebresne
>            Priority: Critical
>              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