qpid-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Keith Wall (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (QPID-6831) Remove logging from constructor of ConnectionScopedRuntimeException
Date Wed, 18 Nov 2015 09:22:10 GMT

    [ https://issues.apache.org/jira/browse/QPID-6831?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15010555#comment-15010555
] 

Keith Wall commented on QPID-6831:
----------------------------------

Alex, exchange already exists and friends etc are actually re-thrown as checked {{MBeanException}}
or {{OperationException}}.  I was not handling this.  I have changed the code to log these
are INFO by default too. I also added the exceptions we treat as a 409 in the HTTP management
layer to the list.  

I will raise a JIRA for the removal of TransportException, but suggest we action this for
6.1.




> Remove logging from constructor of ConnectionScopedRuntimeException
> -------------------------------------------------------------------
>
>                 Key: QPID-6831
>                 URL: https://issues.apache.org/jira/browse/QPID-6831
>             Project: Qpid
>          Issue Type: Improvement
>          Components: Java Broker
>            Reporter: Keith Wall
>            Assignee: Alex Rudyy
>            Priority: Minor
>             Fix For: qpid-java-6.0
>
>
> {{ConnectionScopedRuntimeException}} currently logs itself at ERROR, with a complete
stacktrace, on construction.  The CSRE is intended to be used when the Broker detects an abnormal
situation which should  cause only the affected connection to be closed, for instance, a badly
composed frame.
> The responsibility for logging the CSRE need to be taken higher up the stack, so it can
chose the appropriate level of detail.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org


Mime
View raw message