[ https://issues.apache.org/jira/browse/CASSANDRA-2460?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13019785#comment-13019785 ]
Sylvain Lebresne commented on CASSANDRA-2460: --------------------------------------------- bq. I'd kind of like to push the validation into SSTIdentityIterator, so it gets done in a single place. Is that reasonable? Yes very reasonable, it does simply the code a bit. Attaching v2 that does just that. > Make scrub validate deserialized columns > ---------------------------------------- > > Key: CASSANDRA-2460 > URL: https://issues.apache.org/jira/browse/CASSANDRA-2460 > Project: Cassandra > Issue Type: Improvement > Components: Core > Reporter: Sylvain Lebresne > Assignee: Sylvain Lebresne > Fix For: 0.7.5 > > Attachments: 0001-Make-scrub-check-column-fields-v2.patch, > 0001-Make-scrub-check-column-fields.patch > > Original Estimate: 4h > Remaining Estimate: 4h > > Right now, scrub deserialize the columns but don't validate the fields, and > such there is a number of errors it could fix (or at least corrupted rows it > could skip) but don't. > This ticket proposes to handle those errors. -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira