You are viewing a plain text version of this content. The canonical link for it is here.

Posted to distributedlog-dev@bookkeeper.apache.org by yzang <gi...@git.apache.org> on 2016/07/27 00:39:48 UTC

[GitHub] incubator-distributedlog pull request #5: DL-21: Fix DL flaky test cases

GitHub user yzang opened a pull request:

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

    DL-21: Fix DL flaky test cases

    DL-21: Fixed a few DL flaky test cases.

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

    $ git pull https://github.com/yzang/incubator-distributedlog yzang/fix_flaky_test

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

    https://github.com/apache/incubator-distributedlog/pull/5.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 #5
    
----
commit b87d4069d51c1ba3e1604087a6faddccde6a2d61
Author: Yiming Zang <yz...@twitter.com>
Date:   2016-07-21T01:09:33Z

    fix test cases caused by time sensitive issues
    
    RB_ID=854009

commit 5aaff644b03f72ad9193216b63ceb0bf83cf5127
Author: Yiming Zang <yz...@twitter.com>
Date:   2016-07-22T17:05:02Z

    Fix some flaky tests
    
    RB_ID=854457

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-distributedlog issue #5: DL-21: Fix DL flaky test cases

Posted by sijie <gi...@git.apache.org>.

Github user sijie commented on the issue:

    https://github.com/apache/incubator-distributedlog/pull/5
  
    The change looks good to me. +1.
    
    (regarding the ci check failure, created https://issues.apache.org/jira/browse/DL-22 for tracking the flaky test)


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-distributedlog pull request #5: DL-21: Fix DL flaky test cases

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-distributedlog issue #5: DL-21: Fix DL flaky test cases

Posted by yzang <gi...@git.apache.org>.

Github user yzang commented on the issue:

    https://github.com/apache/incubator-distributedlog/pull/5
  
    This one is quite tricky, I'm not able to reproduce the same problem on my local machine. Maybe due to the different building environment?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---