[CASSANDRA-3559] CFMetaData conversions to Thrift/Native schema should be inverse one of the other

• Type: Improvement
• Status: Resolved
• Priority: Normal
• Resolution: Fixed
• Fix Version/s: 1.1.0
• Labels: avro thrift
• Assignee: Sylvain Lebresne
• Reporter: Sylvain Lebresne

Authors

Sylvain Lebresne

Reviewers

Pavel Yaskevich

• Created: 02/Dec/11 12:14
• Updated: 16/Apr/19 09:32
• Resolved: 25/Jan/12 11:13

Description

In other word, it would probably be a good idea to have:

  cfm == CFMetadata.fromThrift(cfm.toThrift())
  cfm == CFMetadata.fromSchema(cfm.toSchema())

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.