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

Stefania commented on CASSANDRA-9852:
-------------------------------------

h3. Old analysis

Code coverage results after running some relevant dtests for CASSANDRA-9102: 
*consistecy_test.py, paxos_test.py, batch_test.py, counter_test.py and 
secondary_indexes_test.py*.

Quick observations after analyzing StorageProxy and auxiliary files:

||File||Percentage||Missing coverage||
|AbstractReadExecutor|97%|RetryType.ALWAYS|
|ReadCallback|55%|Exception handling or generation (ReadTimeout, ReadFailure, 
DigestMismatch)|
|StorageProxy|56%|Exception handling (timeouts, failures), hinting due to 
exception handling and max limit of hints reached, logged batches and triggers, 
some estimation of ranges due to indexes and range slices, describe cluster (a 
nodetool command), truncation of data, paxos contentions, jmx methods (but I 
think we have some nodetool tests I did not run and they are trivial)|
|AbstractRowResolver|90%|OK|
|RowDigestResolver|88%|OK|
|RowDataResolver|79%|replyCount == 1|
|RangeSliceResponseResolver|19%|Everything|

See [this 
comment|https://issues.apache.org/jira/browse/CASSANDRA-9102?focusedCommentId=14597362&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14597362]
 for more context.

> Complete test coverage for StorageProxy
> ---------------------------------------
>
>                 Key: CASSANDRA-9852
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-9852
>             Project: Cassandra
>          Issue Type: Test
>            Reporter: Stefania
>            Assignee: Stefania
>
> Run relevant unit and dtests under code coverage to find out which parts of 
> StorageProxy and auxiliary classes are still not tested. Below is an old 
> analysis  which should be repeated now that CASSANDRA-8099 is integrated into 
> trunk.
> Then add more unit or distributed tests to complete code coverage. The 
> framework provided by CASSANDRA-9165 will most likely be required to inject 
> failures and time outs.



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

Reply via email to