Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.1
    • Fix Version/s: 4.0
    • Component/s: other
    • Labels:
      None

      Description

      As attached in the jstack, AppServerStartup.stop() hangs in object.wait().

      The problem is discovered while running v3/tests/embedded/basic/lifecycle in a fast multi-core machine. (Should be able to reproduce on the laptop, but you need to build v3/extras/embedded/all in order to run this test).

      The test does actually does glassfish.start() followed by glassfish.stop() with no delay in between.

      As a workaround, in the test, I tried putting some sleep before stop(), and it seems to solve the issue.

      My guess is that the serverThread.run() in AppServerStartup misses the notify sent by speedy stop(), and hence it hangs in wait() forever.

      1. jstack
        11 kB
        Bhavanishankar

        Activity

        Hide
        Bhavanishankar added a comment -

        jstack

        Show
        Bhavanishankar added a comment - jstack
        Hide
        Nazrul added a comment -

        Are we planning to fix this for 3.1 release?

        Show
        Nazrul added a comment - Are we planning to fix this for 3.1 release?
        Hide
        dochez added a comment -

        I think changing the startup thread logic is too risky at this point. This will need to be documented.

        Show
        dochez added a comment - I think changing the startup thread logic is too risky at this point. This will need to be documented.
        Hide
        Tom Mueller added a comment -

        Bulk update to change fix version to "not determined" for all issues still open but with a fix version for a released version.

        Show
        Tom Mueller added a comment - Bulk update to change fix version to "not determined" for all issues still open but with a fix version for a released version.
        Hide
        jwells added a comment -

        This was fixed with checkin 56518 in 4.0

        Show
        jwells added a comment - This was fixed with checkin 56518 in 4.0

          People

          • Assignee:
            Tom Mueller
            Reporter:
            Bhavanishankar
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: