You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Marcus Eriksson (JIRA)" <ji...@apache.org> on 2018/06/01 12:17:00 UTC
[jira] [Commented] (CASSANDRA-14467) Add option to sanity check
tombstones on reads/compaction
[ https://issues.apache.org/jira/browse/CASSANDRA-14467?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16497918#comment-16497918 ]
Marcus Eriksson commented on CASSANDRA-14467:
---------------------------------------------
bq. why is this not just an assertion? It seems very assertion like?
Agree, was debating this a bit, but biggest reason I went this way was to have a way of just logging any validation failures. Also, doing this validation is not very expensive, but it is not free, so you might not want to disable all assertions to just disable this check.
Push a few new commits addressing the latest round of comments
> Add option to sanity check tombstones on reads/compaction
> ---------------------------------------------------------
>
> Key: CASSANDRA-14467
> URL: https://issues.apache.org/jira/browse/CASSANDRA-14467
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Marcus Eriksson
> Assignee: Marcus Eriksson
> Priority: Minor
> Fix For: 4.x
>
>
> We should add an option to do a quick sanity check of tombstones on reads + compaction. It should either log the error or throw an exception.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cassandra.apache.org
For additional commands, e-mail: commits-help@cassandra.apache.org