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/03/22 16:14:22 UTC

[jira] [Updated] (CASSANDRA-4037) Move CfDef and KsDef validation to CFMetaData and KSMetaData

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

Sylvain Lebresne updated CASSANDRA-4037:
----------------------------------------

    Attachment: 0001-Fix-CFMetadata-copyOpts.txt

This patch actually broke the distributed tests because CFMetaData.copyOpts wasn't correctly cloning the columns metadata. Attached fix (against 1.1.1, copyOpts is not used in previous version).
                
> Move CfDef and KsDef validation to CFMetaData and KSMetaData
> ------------------------------------------------------------
>
>                 Key: CASSANDRA-4037
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-4037
>             Project: Cassandra
>          Issue Type: Improvement
>            Reporter: Sylvain Lebresne
>            Assignee: Sylvain Lebresne
>            Priority: Minor
>             Fix For: 1.1.1
>
>         Attachments: 0001-Fix-CFMetadata-copyOpts.txt
>
>
> Following CASSANDRA-3792, CQL don't need to use thrift CfDef and KsDef. However, those are still used in order to reuse ThriftValidation validation methods. We should move that validation to CFM and KSM and remove the use of those thrift structures by CQL.

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