ActiveMQ
  1. ActiveMQ
  2. AMQ-4158

BTreeIndexTest hangs when run on Windows

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 5.7.0
    • Fix Version/s: 5.8.0
    • Component/s: Test Cases
    • Labels:
      None
    • Environment:

      Windows 2008 Server

    • Patch Info:
      Patch Available
    • Regression:
      Unit Test Broken

      Description

      BTreeIndexTest hangs when run on windows because it repeatedly tries to create/delete the same directory, which is problematic on windows. I've attached a patch which just adds a timestamp to the directory name.

        Activity

        Hide
        Timothy Bish added a comment -

        Patch applied, thanks.

        Show
        Timothy Bish added a comment - Patch applied, thanks.
        Hide
        Kevin Earls added a comment -

        Unfortunately my first fix did not work. I will attach another patch which should. It converts several tests to JUnit4, add's timeouts in the @Test annotation, and moves the code that I think was causing the hang out of setup()

        Show
        Kevin Earls added a comment - Unfortunately my first fix did not work. I will attach another patch which should. It converts several tests to JUnit4, add's timeouts in the @Test annotation, and moves the code that I think was causing the hang out of setup()
        Hide
        Kevin Earls added a comment -

        Converted tests to JUnit 4 and added timeouts to prevent windows hangs

        Show
        Kevin Earls added a comment - Converted tests to JUnit 4 and added timeouts to prevent windows hangs
        Hide
        Timothy Bish added a comment -

        Unfortunately the patch is not able to be applied cleanly with the recent refactorings in trunk.

        Show
        Timothy Bish added a comment - Unfortunately the patch is not able to be applied cleanly with the recent refactorings in trunk.
        Hide
        Timothy Bish added a comment -

        Reworked the patch so it would apply to the new layout, changes added in trunk.

        Show
        Timothy Bish added a comment - Reworked the patch so it would apply to the new layout, changes added in trunk.

          People

          • Assignee:
            Unassigned
            Reporter:
            Kevin Earls
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development