ActiveMQ
  1. ActiveMQ
  2. AMQ-2511

Regression: Inactivity monitor does not time out stale connections.

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 5.3.0
    • Fix Version/s: 5.3.2, 5.4.0
    • Component/s: Broker
    • Labels:
      None
    • Regression:
      Regression

      Description

      The fix @ AMQ-2088 cause a the Inactivity Monitor to regress.

      Now when a socket is waiting to read the next message, or is in the middle of receiving the next message it sets a flag that disable the inactivity monitor from timing out the connection. The problem is that if the connection gets hung, then it won't time out since the flag is set.

      I think a better approach would be for the wireformat to expose a counter which is incremented as data is read.

      That way between inactivity check intervals the inactivity monitor can check to see if that value has changed since it's last check.

        Issue Links

          Activity

          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          169d 15h 19m 1 Gary Tully 14/May/10 09:41
          Jeff Turner made changes -
          Project Import Fri Nov 26 22:32:02 EST 2010 [ 1290828722158 ]
          Gary Tully made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Fix Version/s 5.3.2 [ 12310 ]
          Resolution Fixed [ 1 ]
          Hide
          Gary Tully added a comment -

          fix is also on 5.3 branch

          Show
          Gary Tully added a comment - fix is also on 5.3 branch
          Hide
          Hiram Chirino added a comment -

          fix applied to 5.4 branch in rev 884267.
          need to back port to 5.3 branch.

          Show
          Hiram Chirino added a comment - fix applied to 5.4 branch in rev 884267. need to back port to 5.3 branch.
          Hiram Chirino made changes -
          Field Original Value New Value
          Link This issue is broken by AMQ-2088 [ AMQ-2088 ]
          Hiram Chirino created issue -

            People

            • Assignee:
              Hiram Chirino
              Reporter:
              Hiram Chirino
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development