Details

    • Sub-task
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 0.15
    • REEF.NET
    • None

    Description

      TcpConnectionRetryTest.TestConnectionRetries passes when executed from VS but fails (reproducible) when tests are executed from command line.

      Expected # of retries is 5, but 0 are found by the code. Test log shows messages INFO: Retry - Count:5, Delay:00:00:00.5000000, ..., so it's a matter of test not being able to find the messages in the output, not of retries not done.

      Also, assert.equals needs to have a clear message, currently actual and expected values are swapped.

      Repros: Markus Weimer, Mariia Mykhailova.

      Attachments

        Activity

          Dhruv Mahajan, please take a look.

          MariiaMykhailova Mariia Mykhailova added a comment - Dhruv Mahajan , please take a look.
          dkm2110 Dhruv Mahajan added a comment - Mariia Mykhailova Markus Weimer please find the pull request for the fix. https://github.com/apache/reef/pull/935
          markus.weimer Markus Weimer added a comment -

          Resolved via #935

          markus.weimer Markus Weimer added a comment - Resolved via #935

          People

            dkm2110 Dhruv Mahajan
            MariiaMykhailova Mariia Mykhailova
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Error getting Slack data from Jira

                Issue deployment

                  Client must be authenticated to access this resource.