[
https://issues.apache.org/jira/browse/CASSANDRA-3559?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Sylvain Lebresne updated CASSANDRA-3559:
----------------------------------------
Description:
In other word, it would probably be a good idea to have:
{noformat}
cfm == CFMetadata.fromThrift(cfm.toThrift())
cfm == CFMetadata.fromSchema(cfm.toSchema())
{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.
was:
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.
Priority: Major (was: Minor)
Summary: CFMetaData conversions to Thrift/Native schema should be
inverse one of the other (was: CFMetaData conversions to Thrift/Avro should
probably be inverse one of the other)
> CFMetaData conversions to Thrift/Native schema should 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
> Labels: avro, thrift
> Fix For: 1.1
>
> Attachments: 3559.patch, CASSANDRA-3559-v2.patch
>
>
> In other word, it would probably be a good idea to have:
> {noformat}
> cfm == CFMetadata.fromThrift(cfm.toThrift())
> cfm == CFMetadata.fromSchema(cfm.toSchema())
> {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.
--
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
