Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: M2.1
    • Fix Version/s: 0.15
    • Component/s: Broker-J
    • Labels:
      None

      Description

      Firstly need to define Exception handling policies and apply consistently, then need to look at specific resolution for the two linked JIRAs too.

      The Java broker does not define or implement any form of consistent exception handling. In particular unrecoverable erros are at best handled by logging and then attempting to carry on regardless.

      I would argue that such errors should be followed by an immediate System.exit ... however others may wish this to be configurable.

        Issue Links

          Activity

          Hide
          gemmellr Robbie Gemmell added a comment -

          Looks good to me.

          Show
          gemmellr Robbie Gemmell added a comment - Looks good to me.
          Hide
          rgodfrey Rob Godfrey added a comment -

          Robbie can you review this please

          Show
          rgodfrey Rob Godfrey added a comment - Robbie can you review this please
          Hide
          k-wall Keith Wall added a comment -

          I think the new uncaught exception handler should also try to log the exception to the Broker log file (using log4j) in addition to stderr. I say this as a) our users are already accustomed to checking this log file b) stderr many be swallowed by job control system or lost through careless redirection.

          Show
          k-wall Keith Wall added a comment - I think the new uncaught exception handler should also try to log the exception to the Broker log file (using log4j) in addition to stderr. I say this as a) our users are already accustomed to checking this log file b) stderr many be swallowed by job control system or lost through careless redirection.
          Hide
          aidan Aidan Skinner added a comment -

          Bagsie not it

          Show
          aidan Aidan Skinner added a comment - Bagsie not it
          Hide
          marnie Marnie McCormack added a comment -

          Moving items not being worked on afaik out of M4 Fix Version

          Show
          marnie Marnie McCormack added a comment - Moving items not being worked on afaik out of M4 Fix Version

            People

            • Assignee:
              gemmellr Robbie Gemmell
              Reporter:
              rgodfrey Rob Godfrey
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development