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

Brandon Williams commented on CASSANDRA-18189:
----------------------------------------------

Given that this is not a new regression for 5.0 I'm going to stop working on it 
for now.  Here are my notes:

The [returns from 
cqlsh|https://github.com/apache/cassandra-dtest/blob/trunk/cqlsh_tests/test_cqlsh_copy.py#L2422]
 are collected but never inspected.  The number of records reported in the log 
after this is just 
[regurgitated|https://github.com/apache/cassandra-dtest/blob/trunk/cqlsh_tests/test_cqlsh_copy.py#L2423]
 from the original insert, so it can easily be incorrect.  It is ignored errors 
here that result in the math being broken for the assertion later.  This can 
affect all versions of the _bulk_round_trip_ tests.

The _with_timeouts_ version specifically also has the issue of relying on small 
read/write request timeout settings to force timeouts to occur, which will 
obviously be very sensitive to the environment the test is run in.

> Test failure in 
> cqlsh_tests.test_cqlsh_copy.TestCqlshCopy.test_bulk_round_trip_with_timeouts
> --------------------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-18189
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-18189
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Test/dtest/python
>            Reporter: Caleb Rackliffe
>            Assignee: Brandon Williams
>            Priority: Normal
>              Labels: cqlsh, dtest, python, test
>             Fix For: 3.0.x, 3.11.x, 4.0.x, 4.1.x, 5.x
>
>
> https://ci-cassandra.apache.org/job/Cassandra-4.0/523/testReport/dtest.cqlsh_tests.test_cqlsh_copy/TestCqlshCopy/test_bulk_round_trip_with_timeouts/
> {noformat}
> assert 100000 == 94764
>  +  where 100000 = sum(<generator object 
> TestCqlshCopy._test_bulk_round_trip.<locals>.<genexpr> at 0x7f8b21987b90>)
>  +  and   94764 = sum(<generator object 
> TestCqlshCopy._test_bulk_round_trip.<locals>.<genexpr> at 0x7f8b21986dc0>)
> {noformat}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

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

Reply via email to