You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Benedict (JIRA)" <ji...@apache.org> on 2015/02/20 11:01:11 UTC

[jira] [Created] (CASSANDRA-8840) Classify our Assertions (like com.google.base.Preconditions)

Benedict created CASSANDRA-8840:
-----------------------------------

             Summary: Classify our Assertions (like com.google.base.Preconditions)
                 Key: CASSANDRA-8840
                 URL: https://issues.apache.org/jira/browse/CASSANDRA-8840
             Project: Cassandra
          Issue Type: Improvement
          Components: Core
            Reporter: Benedict
            Priority: Minor


I raised this on IRC, then dropped it due to opposition, but it's possible the opposition was due to my conflation of the act of classification with the disabling of some of the assertions. The two aren't wed, and I think it _would_ improve readability significantly by itself, as Ariel reminded me with his use of google's Preconditions class in CASSANDRA-8692.

I would prefer to use our own version of this class, that we can force the \@Inline compiler hint onto, so that we have no negative performance implications. Also, we can then introduce a class of data corruption checks, etc. I think this would aid readability, and also permit easier analysis of the codebase via IDE (right now it's very hard to say what data corruption checks we actually perform, for instance).

Thoughts?



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)