reef-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Douglas Service <>
Subject Re: Pull request blocked by failing tests from REEF-1473
Date Thu, 29 Sep 2016 20:43:47 GMT
Hello Mariia,

The failures were in visual studio. I would have sent the errors last night
but I could not find any way to export the results from visual studio. An
png is attached that shows some of the errors. I also included the output
from git log.  Do you know how to export test results from VisualStudio? I
will try a build and test from the command line and if  that succeeds, I
will go ahead with the pull request.


commit 75f78ac8dfbcfa76bf4bde82948761d0fd1372db
Author: Mariia Mykhailova <>
Date:   Wed Sep 28 15:20:02 2016 -0700

    [REEF-1473] Add check for strStatus=null to TestBridgeClient

    This change adds check for strStatus being null in TestBridgeClient.
    It doesn't fix the root cause of failures, which can't be reproduced
    but slightly improves troubleshooting in case the failures repeat.


    Pull request:
      This closes #1139

commit 6ef63c19633554b3889c14aa203b8c628df4538e
Author: Mariia Mykhailova <>
Date:   Wed Sep 28 12:30:19 2016 -0700

    [REEF-1618] Add stack trace to exceptions handled in TaskHostBase

    This change adds full exception info to log printed in


    Pull request:
      This closes #1138

commit a2278b4db74f6b60b41f1b4c1e848edaa3453de5
Author: Mariia Mykhailova <>
Date:   Tue Sep 27 17:36:13 2016 -0700

    [REEF-1623] Remove double-logging exception in Logger.Log(Exception)

    This change
     * removes logging of exception.Message and exception.StackTrace
       which are logged as part of exception anyways.
     * cleans up logic of when message is logged.


    Pull request:
      This closes #1137

commit 3ea89be97c20a440bba84c8c33338ee8e480875d
Author: Mariia Mykhailova <>
Date:   Mon Sep 26 11:53:29 2016 -0700

    [REEF-1451] Clean up IMRU Fault Tolerant scenario tests

    This change:
     * extends failures simulated with FailureType to include dispose.
     * adds tests for failure of mapper task on init and on dispose
       and failure of mapper evaluator on dispose.
     * cleans up checks and comments in other IMRU FT scenario tests.


[image: Inline image 1]

On Thu, Sep 29, 2016 at 11:10 AM, Mariia Mykhailova <>

> Doug, you probably want to share the errors you're getting in the tests. I
> don't think REEF-1473 should cause any test failures, we've tested it
> locally, and it also passes AppVeyor build (
> project/ApacheSoftwareFoundation/reef/build/1000-master)
> -Mariia
> -----Original Message-----
> From: Markus Weimer []
> Sent: Wednesday, September 28, 2016 10:28 PM
> To: REEF Developers Mailinglist <>
> Subject: Re: Pull request blocked by failing tests from REEF-1473
> I have just tested both current `master` as well as your withdrawn PR
> #1140. Both pass tests for me. Could it be that the issue is local on your
> machine?
> Markus
> On Wed, Sep 28, 2016 at 9:21 PM, Markus Weimer <> wrote:
> > That is odd, I tested that PR before merging it. What's the error
> > message of the tests that failed?
> >
> > Markus
> >
> > On Wed, Sep 28, 2016 at 7:34 PM, Douglas Service <>
> wrote:
> >> I have a pull request ready to go for REEF-1611 but cannot push it do
> >> a failing test from REEF-1473. I verified the problem on a clean
> >> branch with a pull on upstream master.  A number of tests in
> >> Org.Apache.REEF.Tests.Functional.Group are failing and it appears to
> >> be due to changes in TestBridgeClient.cs.
> >>
> >> Doug

  • Unnamed multipart/related (inline, None, 0 bytes)
View raw message