Uploaded image for project: 'ActiveMQ Artemis'
  1. ActiveMQ Artemis
  2. ARTEMIS-5206

Provide description = "" instead of null in certain exception instances

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Trivial
    • Resolution: Fixed
    • 2.38.0
    • 2.39.0
    • None
    • None

    Description

      Certain exception types, for example ActiveMQRemoteDisconnectException are showing the following exception trace when thrown:

      org.apache.activemq.artemis.api.core.ActiveMQRemoteDisconnectException: null
      	at org.apache.activemq.artemis.core.remoting.server.impl.RemotingServiceImpl.connectionDestroyed(RemotingServiceImpl.java:600) ~[classes/:?]
      	at org.apache.activemq.artemis.core.remoting.impl.invm.InVMAcceptor$Listener.connectionDestroyed(InVMAcceptor.java:296) ~[classes/:?]
      	at org.apache.activemq.artemis.core.remoting.impl.invm.InVMConnection.internalClose(InVMConnection.java:157) ~[classes/:?]
      	at org.apache.activemq.artemis.core.remoting.impl.invm.InVMConnection.disconnect(InVMConnection.java:166) ~[classes/:?]
      	at org.apache.activemq.artemis.core.remoting.impl.invm.InVMAcceptor.disconnect(InVMAcceptor.java:246) ~[classes/:?]
      	at org.apache.activemq.artemis.core.remoting.impl.invm.InVMConnector$Listener.connectionDestroyed(InVMConnector.java:274) ~[classes/:?]
      	at org.apache.activemq.artemis.core.remoting.impl.invm.InVMConnection.internalClose(InVMConnection.java:157) ~[classes/:?]
      	at org.apache.activemq.artemis.core.remoting.impl.invm.InVMConnection.close(InVMConnection.java:145) ~[classes/:?]
      

      It would be more aesthetically if they were providing "" as a description instead of "null"

      This caught my attention on logs as I thought it was a NullPointerException.

      This is just a trivial change.

      Attachments

        Activity

          People

            clebertsuconic Clebert Suconic
            clebertsuconic Clebert Suconic
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: