[ https://issues.apache.org/jira/browse/CASSANDRA-3559?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Sylvain Lebresne updated CASSANDRA-3559: ---------------------------------------- Attachment: 3559.patch Note that this actually tests CASSANDRA-3558 and hence the uni test won't pass without it. > CFMetaData conversions to Thrift/Avro should probably be inverse one of the > other > --------------------------------------------------------------------------------- > > Key: CASSANDRA-3559 > URL: https://issues.apache.org/jira/browse/CASSANDRA-3559 > Project: Cassandra > Issue Type: Improvement > Components: Core > Reporter: Sylvain Lebresne > Assignee: Sylvain Lebresne > Priority: Minor > Attachments: 3559.patch > > > In other word, it would probably be a idea to have: > {noformat} > cfm == CFMetadata.fromThrift(cfm.toThrift()) > cfm == CFMetadata.fromAvro(cfm.toAvro()) > {noformat} > In particular, we could have unit tests to check that, which would avoid > things like CASSANDRA-3558. > It is not the case today for thrift because of the keyAlias. For some reason, > if the keyAlias is not set, we return with toThrift() the default alias. I > don't think this serves any purpose though. > The goal of this ticket is to both fix that (unless there is a compelling > reason not to) and add unit tests for this. -- 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