[ https://issues.apache.org/jira/browse/CASSANDRA-1480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12908924#action_12908924 ]
Jon Hermes edited comment on CASSANDRA-1480 at 9/13/10 2:58 PM: ---------------------------------------------------------------- convertToCFMetaData had to be changed as well, it was expecting subcomparator to default to empty string. I don't think this was a recent change; instead it looks more like we changed our assumption about the default. In any event, subcomp, reconciler, and comment all default to null now and all tests/methods are working correctly. was (Author: jhermes): convertToCFMetaData had to be changed as well, it was expecting subcomparator to default to empty string. I don't think this was a recent change; instead it looks more like we changed our assumpting about the default. In any event, subcomp, reconciler, and comment all default to null now and all tests/methods are working correctly. > CFMetaData.convertToThrift makes subcomparator_type empty string instead of > null > -------------------------------------------------------------------------------- > > Key: CASSANDRA-1480 > URL: https://issues.apache.org/jira/browse/CASSANDRA-1480 > Project: Cassandra > Issue Type: Bug > Components: API > Affects Versions: 0.7 beta 1 > Reporter: Jeremy Hanna > Assignee: Jon Hermes > Fix For: 0.7 beta 2 > > Attachments: 1480.txt > > > As a result of CASSANDRA-891 adding a CFMetaData.convertToThrift method, the > values such as subcomparator_type are defaulted to empty string instead of > null. That makes it so, for example, in ColumnFamilyRecordReader, in its > RowIterator, the check for only null is insufficient. It also needs to check > for a blank value. > After a discussion about it in IRC, Jonathan said it was probably easier to > just change the creation to give a null value instead of empty string. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.