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

ASF GitHub Bot commented on ARTEMIS-1472:
-----------------------------------------

Github user tabish121 commented on the issue:

    https://github.com/apache/activemq-artemis/pull/1601
  
    I wouldn't tie together all the bits into this one env flag, most often 
logging the binary stuff adds nothing other than noise in all but some very 
specific cases.  To be honest activating logging based on env vars seems weird 
in a modern Java app where we have fancy things like loggers that can be 
configured in config files specific to the running application.  


> Use PN_TRACE_FRAME as a signal for System.out logging on AMQP events
> --------------------------------------------------------------------
>
>                 Key: ARTEMIS-1472
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-1472
>             Project: ActiveMQ Artemis
>          Issue Type: Improvement
>          Components: AMQP
>    Affects Versions: 2.3.0
>            Reporter: clebert suconic
>            Assignee: clebert suconic
>             Fix For: 2.4.0
>
>
> AMQP developers will usually set PN_TRACE_FRM as a System.property to get out 
> some information from Proton regarding the frame processing.
> It would be a nice idea to also throw some log.info when that property is set 
> at Artemis.. around AMQP Processing, making it easier to Debug AMQP packets.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to