Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: M2.1
    • Fix Version/s: 0.15
    • Component/s: Java Broker
    • 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
          Marnie McCormack added a comment -

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

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

          Bagsie not it

          Show
          Aidan Skinner added a comment - Bagsie not it
          Hide
          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
          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
          Rob Godfrey added a comment -

          Robbie can you review this please

          Show
          Rob Godfrey added a comment - Robbie can you review this please
          Hide
          Robbie Gemmell added a comment -

          Looks good to me.

          Show
          Robbie Gemmell added a comment - Looks good to me.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development