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

Benjamin Lerer commented on CASSANDRA-16192:
--------------------------------------------

[~Bereng] The problem I see is that you cannot have reliable tests unless your 
expectations are clear. Right now we do not seems to have a clear definition of 
what some metric output should be and they will differ based on the strategy 
you use. The current PR used a sub-class of {{SizeTieredCompactionStrategy}} 
for testing the metrics, ignoring the fact that the metrics do not seems work 
in the same way for the others strategies. For the tests to be reliable it 
seems to me that the expected output should be well defined and that the test 
should test the 3 main compaction strategies. 

> Add more tests to cover compaction metrics
> ------------------------------------------
>
>                 Key: CASSANDRA-16192
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-16192
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Test/unit
>            Reporter: Benjamin Lerer
>            Assignee: Adam Holmberg
>            Priority: Normal
>             Fix For: 4.0-rc
>
>
> Some compaction metrics do not seems to be tested.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to