You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Sylvain Lebresne (Updated) (JIRA)" <ji...@apache.org> on 2012/01/25 17:38:41 UTC

[jira] [Updated] (CASSANDRA-3753) Update CqlPreparedResult to provide type information

     [ https://issues.apache.org/jira/browse/CASSANDRA-3753?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Sylvain Lebresne updated CASSANDRA-3753:
----------------------------------------

    Attachment: 0002-Thrift-gen-file-changes.patch
                0001-3753-cql3.patch

Attaching patch for cql3. Most of the patch is a slight refactor of the class hierarchy to make all this fit better.

For the CqlPrepareResult, this adds an optional list<string> of types. With that, the count is actually redundant, but I've kept it for compatibility with cql2, as it's unclear we'll be able to do the same thing.

I don't think drivers have much support of prepared statement so far so I didn't find a clean way to test this. But I did make a manual simple test to check it does return the right types in the right order.
                
> Update CqlPreparedResult to provide type information
> ----------------------------------------------------
>
>                 Key: CASSANDRA-3753
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3753
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: API
>    Affects Versions: 1.1
>            Reporter: Jonathan Ellis
>            Priority: Critical
>              Labels: cql
>             Fix For: 1.1
>
>         Attachments: 0001-3753-cql3.patch, 0002-Thrift-gen-file-changes.patch
>
>
> As discussed on CASSANDRA-3634, adding type information to a prepared statement would allow more client-side error checking.

--
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