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

Sylvain Lebresne commented on CASSANDRA-4037:
---------------------------------------------

Pushed patch to https://github.com/pcmanus/cassandra/commits/4037

This move validation to CFMetaData and KSMetaData and the remove any now 
useless conversion to/from thrift. The patch is against cassandra-1.1 and to be 
honest it moves quite a bit of code around so even though that's a direct 
follow up of CASSANDRA-3792, there is probably no point pushing this to 1.1.0.
                
> 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
>
>
> 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

        

Reply via email to