ICEfaces
  1. ICEfaces
  2. ICE-7988

Hanging Monitor Runner thread upon shutdown or redeploy

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-1.8.2.GA_P03
    • Fix Version/s: EE-1.8.2.GA_P04
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      ICEfaces

      Description

      This issue is related to ICE-6967 and deals with the Monitor Runner thread.

        Issue Links

          Activity

          Hide
          Jack Van Ooststroom added a comment -

          Sending src/com/icesoft/util/MonitorRunner.java
          Transmitting file data .
          Committed revision 28627.

          Show
          Jack Van Ooststroom added a comment - Sending src/com/icesoft/util/MonitorRunner.java Transmitting file data . Committed revision 28627.
          Hide
          Jack Van Ooststroom added a comment -

          The logic in the spawned thread was being executed every 10s using a Thread.sleep(). Setting the run boolean to false while shutting down, didn't necessarily mean the thread got stopped before Tomcat completed the shutdown, hence the warning. When the run boolean is set to false, it now tries to join the spawned thread with a timeout of 1000ms. In addition the logic is still executed every 10s, but the loop is "interrupted" every 1s to check to see if the run boolean is still set to true. This avoids the join logic to take up 10s. Marking this one as FIXED.

          Show
          Jack Van Ooststroom added a comment - The logic in the spawned thread was being executed every 10s using a Thread.sleep(). Setting the run boolean to false while shutting down, didn't necessarily mean the thread got stopped before Tomcat completed the shutdown, hence the warning. When the run boolean is set to false, it now tries to join the spawned thread with a timeout of 1000ms. In addition the logic is still executed every 10s, but the loop is "interrupted" every 1s to check to see if the run boolean is still set to true. This avoids the join logic to take up 10s. Marking this one as FIXED.
          Hide
          Jack Van Ooststroom added a comment -

          Sending core/src/com/icesoft/util/MonitorRunner.java
          Transmitting file data .
          Committed revision 28655.

          Show
          Jack Van Ooststroom added a comment - Sending core/src/com/icesoft/util/MonitorRunner.java Transmitting file data . Committed revision 28655.

            People

            • Assignee:
              Jack Van Ooststroom
              Reporter:
              Jack Van Ooststroom
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: