[jira] [Commented] (KUDU-2109) TabletCopyClientSessionITest.TestCopyFromCrashedSource is flaky

["Todd Lipcon (JIRA)" <ji...@apache.org>]

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

Todd Lipcon commented on KUDU-2109:
-----------------------------------

For better searchability, the error message looks like:
/data/somelongdirectorytoavoidrpathissues/src/kudu/src/kudu/integration-tests/tablet_copy_client_session-itest.cc:273
Failed
Bad status: Already present: State transition of tablet 13de602c5d72429eac4ee2a9f547f481 already in progress: copying tablet


> TabletCopyClientSessionITest.TestCopyFromCrashedSource is flaky
> ---------------------------------------------------------------
>
>                 Key: KUDU-2109
>                 URL: https://issues.apache.org/jira/browse/KUDU-2109
>             Project: Kudu
>          Issue Type: Bug
>          Components: tserver
>    Affects Versions: 1.4.0
>            Reporter: Adar Dembo
>            Assignee: Todd Lipcon
>            Priority: Major
>         Attachments: 0_tablet_copy_client_session-itest.txt
>
>
> I've attached the full log from my test failure.
> I think I've found the issue too: the test assumes that if it finds an on-disk superblock in the TOMBSTONED state, the failed tablet copy has finished and it's safe to start another one. However, in TSTabletManager::RunTabletCopy, 'tc_client' goes out of scope before 'deleter', which means that the TabletCopyClient destructor (which deletes the on-disk data, flushing the superblock in the TOMBSTONED state) will run before the TransitionInProgress destructor (which removes the tablet's ID from the global map tracking transitions, allowing a new tablet copy to proceed).



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)