You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Gösta Forsum (JIRA)" <ji...@apache.org> on 2013/11/26 09:48:37 UTC

[jira] [Commented] (CASSANDRA-6400) Update unit tests to use latest Partitioner

    [ https://issues.apache.org/jira/browse/CASSANDRA-6400?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13832413#comment-13832413 ] 

Gösta Forsum commented on CASSANDRA-6400:
-----------------------------------------

Why ?

Changing to Murmur3Partitioner casues approx 35 unit tests to fail. 
Examining some of failing unit test reveals that they rely heavily on the fact the ByteOrderedPartitioner is used. 

Changing to Murmur3Partitioner would increase the complexity of these tests.
(For example: StorageProxyTest.testGRR)

there are separate tests for Murmur3Partitioner.

Testing complicated combinations of classes is more a integration test task.
It is better to have simple unit tests testing the parts and then a few integration tests that try likely combinations of them.

just my 2 cents



> Update unit tests to use latest Partitioner
> -------------------------------------------
>
>                 Key: CASSANDRA-6400
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-6400
>             Project: Cassandra
>          Issue Type: Test
>          Components: Tests
>            Reporter: Alex Liu
>            Priority: Minor
>
> test/conf/cassandra.yaml uses the out-date ByteOrderedPartitioner, we should update it to Murmur3Partitioner.



--
This message was sent by Atlassian JIRA
(v6.1#6144)