[ 
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

Reply via email to