Qpid
  1. Qpid
  2. QPID-3187

No exception handling for many timer tasks in broker

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.8, 0.10
    • Fix Version/s: 0.11
    • Component/s: None
    • Labels:
      None

      Description

      The assumption in Timer appears to be that no thread will ever throw from TimerTask::fire(). In practice however few of the implementations of that method in the broker take any measures to ensure that is the case. If code called on a timer thread throws there is therefore the risk of the broker terminating (I have observed such a case for example on the management periodic processing).

      It would probably be advisable to add some generic protection for this.

        Activity

        Hide
        Gordon Sim added a comment -

        Patch submitted for review at https://reviews.apache.org/r/546/

        Show
        Gordon Sim added a comment - Patch submitted for review at https://reviews.apache.org/r/546/

          People

          • Assignee:
            Gordon Sim
            Reporter:
            Gordon Sim
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development