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

Joshua McKenzie commented on CASSANDRA-10771:
---------------------------------------------

The interface on Transactional.abort expects a throwable:

{code}
    Throwable abort(Throwable accumulate);
{code}

Rather than passing abort null and then checking for null, would it make more 
sense to just pass e?

{code}
        catch (Throwable e)
        {
            if (writer != null)
            {
                Throwable e2 = writer.abort(null); // pass e here instead of 
null
                // add abort error to original and continue so we can drain 
unread stream
                e.addSuppressed(e2);
            }
{code}

> bootstrap_test.py:TestBootstrap.resumable_bootstrap_test is failing
> -------------------------------------------------------------------
>
>                 Key: CASSANDRA-10771
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-10771
>             Project: Cassandra
>          Issue Type: Sub-task
>          Components: Streaming and Messaging
>            Reporter: Philip Thompson
>            Assignee: Yuki Morishita
>             Fix For: 3.0.1, 3.1
>
>
> When running {{bootstrap_test.py:TestBootstrap.resumable_bootstrap_test}} 
> locally, the test is failing on cassandra-3.0. When I bisect the failure, I 
> find that 87f5e2e39c100, the commit that merged CASSANDRA-10557 into 3.0 is 
> the first failing commit. I can reproduce this consistently locally, but 
> cassci is only having intermittent failures.



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

Reply via email to