[jira] [Commented] (DL-165) Add TimedOutTestsListener to print the full thread dump on test timeout

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

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

ASF GitHub Bot commented on DL-165:
-----------------------------------

GitHub user xieliang opened a pull request:

    https://github.com/apache/incubator-distributedlog/pull/91

    DL-165 : Add TestTimedOutTestsListener to dump timed out cases thread dump

    

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/xieliang/incubator-distributedlog DL-165-TimedOutTestsListener

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-distributedlog/pull/91.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #91
    
----
commit 378e0745d72a91fd91d37c974c60476fa53b91f6
Author: xieliang <xi...@gmail.com>
Date:   2016-12-29T10:23:52Z

    Add TestTimedOutTestsListener to dump timed out cases thread dump

----


> Add TimedOutTestsListener to print the full thread dump on test timeout
> -----------------------------------------------------------------------
>
>                 Key: DL-165
>                 URL: https://issues.apache.org/jira/browse/DL-165
>             Project: DistributedLog
>          Issue Type: Improvement
>          Components: tests
>    Affects Versions: 0.4.0
>            Reporter: Liang Xie
>            Assignee: Liang Xie
>
> The TestBKSyncLogReader.testReadRecordsAfterReadAheadCaughtUp case seems failed frequently due to timeout, i.e. https://builds.apache.org/job/distributedlog-precommit-pullrequest/com.twitter$distributedlog-core/56/testReport/junit/com.twitter.distributedlog/TestBKSyncLogReader/testReadRecordsAfterReadAheadCaughtUp/
> From the log, it's hard to figure out the root cause, it may be helpful if we had the full thread dump. we can add a TimedOutTestsListener just like netty/hadoop projects, please refer to : https://github.com/junit-team/junit4/issues/463 , https://github.com/netty/netty/pull/4749/files , https://issues.apache.org/jira/browse/HADOOP-8755 for more details



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