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

Stefania commented on CASSANDRA-10555:
--------------------------------------

The problem can only be reproduced if {{testSinglePartitionNamesAbort}} runs 
before {{testSinglePartitionSliceAbort}} and it's because of the typo "cdd" 
instead of "dd". So the typo must be fixed. We could write the data only once 
since it is identical but truncating is preferred in case later on we want to 
modify the test methods.

+1 once the test is confirmed to pass on CI as well:

http://cassci.datastax.com/job/jeffjirsa-cassandra-10555-testall/lastBuild/
http://cassci.datastax.com/job/jeffjirsa-cassandra-10555-dtest/lastBuild/

Thank you for fixing this.

> ReadCommandTest should truncate between test cases 
> ---------------------------------------------------
>
>                 Key: CASSANDRA-10555
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-10555
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Tests
>            Reporter: Jeff Jirsa
>            Assignee: Jeff Jirsa
>            Priority: Minor
>
> ReadCommandTest writes to the same CF in multiple unit tests, and then counts 
> the results in a partition. This can lead to tests failing in some 
> circumstances:
> {noformat}
>     [junit] Testcase: 
> testSinglePartitionNamesAbort(org.apache.cassandra.db.ReadCommandTest): FAILED
>     [junit] expected:<2> but was:<1>
>     [junit] junit.framework.AssertionFailedError: expected:<2> but was:<1>
>     [junit]   at 
> org.apache.cassandra.db.ReadCommandTest.testSinglePartitionNamesAbort(ReadCommandTest.java:140)
>     [junit] 
>     [junit] 
>     [junit] Testcase: 
> testSinglePartitionSliceAbort(org.apache.cassandra.db.ReadCommandTest): FAILED
>     [junit] expected:<2> but was:<3>
>     [junit] junit.framework.AssertionFailedError: expected:<2> but was:<3>
>     [junit]   at 
> org.apache.cassandra.db.ReadCommandTest.testSinglePartitionSliceAbort(ReadCommandTest.java:111)
> {noformat}
> This can be fixed trivially by truncating the CF at the beginning of 
> {{testSinglePartitionSliceAbort}} and {{testSinglePartitionNamesAbort}} 
> Also, once the truncate is in place, a potential typo is exposed in 
> {{testSinglePartitionNamesAbort}}, where the partition is written with 
> clustering column {{cc}} and {{cdd}}, but the read command reads {{cc}} and 
> {{dd}} 



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

Reply via email to