ICEpush
  1. ICEpush
  2. PUSH-249

Improve exception logging in the various MessageService classes

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-3.2.0.GA
    • Fix Version/s: EE-3.3.0.GA
    • Component/s: Push Server
    • Labels:
      None
    • Environment:
      ICEpush-EE, EPS

      Description

      Currently when an exception occurs using a publish invocation, and not a publishNow invocation, it is hard to determine the invokee when the QueueMessagePublisher executes the task.

        Activity

        Hide
        Jack Van Ooststroom added a comment -

        Sending src/main/java/com/icesoft/net/messaging/MessageServiceException.java
        Sending src/main/java/com/icesoft/net/messaging/QueueMessagePublisher.java
        Transmitting file data ..
        Committed revision 34579.

        Show
        Jack Van Ooststroom added a comment - Sending src/main/java/com/icesoft/net/messaging/MessageServiceException.java Sending src/main/java/com/icesoft/net/messaging/QueueMessagePublisher.java Transmitting file data .. Committed revision 34579.
        Hide
        Jack Van Ooststroom added a comment -

        The MessageServiceException now extends RuntimeException instead of Exception, which allows it to be thrown from a Runnable.run() method. This exception is now used to wrap the original exception that occurred during a publish invocation and includes the details in its message explaining what was trying to be published. This should help determining the culprit when one of these exceptions is thrown. Marking this one as FIXED.

        Show
        Jack Van Ooststroom added a comment - The MessageServiceException now extends RuntimeException instead of Exception, which allows it to be thrown from a Runnable.run() method. This exception is now used to wrap the original exception that occurred during a publish invocation and includes the details in its message explaining what was trying to be published. This should help determining the culprit when one of these exceptions is thrown. Marking this one as FIXED.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: