ActiveMQ LoggingBrokerPlugin too verbose

[armsargis_sargis]

Hi all, I am using Apache ActiveMQ v5.4.2-fuse-02-00 and I want to log events related connections and added following line into my config

but I now in my log I have a lot of noise because its enabled also events related to sessions. I've prepared patch to add additional flag 'logSessionEvents'  to have more control what to log. Here is pull request URL: https://github.com/apache/activemq/pull/1 and here is log snippet.

2011-03-23 12:58:55,861 | INFO  | Adding Connection : ConnectionInfo {commandId = 1, responseRequired = true, connectionId = ID:comunalserver-3451-634364032937500000-1:1141, clientId = ID:comunalserver-3451-634364032937500000-1142:0, userName = epaybroker, password = *****, brokerPath = null, brokerMasterConnector = false, manageable = false, clientMaster = false, faultTolerant = false} | org.apache.activemq.broker.util.LoggingBrokerPlugin | ActiveMQ Transport: ssl:///81.89.210.236:2972

2011-03-23 12:58:55,903 | INFO  | Adding Session : SessionInfo {commandId = 2, responseRequired = false, sessionId = ID:comunalserver-3451-634364032937500000-1:1141:1} | org.apache.activemq.broker.util.LoggingBrokerPlugin | ActiveMQ Transport: ssl:///81.89.210.236:2972

2011-03-23 12:58:56,517 | INFO  | Removing Session : SessionInfo {commandId = 71563, responseRequired = false, sessionId = ID:test.customs.am-28506-1300862663044-0:2:15904} | org.apache.activemq.broker.util.LoggingBrokerPlugin | ActiveMQ Transport: ssl:///192.168.0.30:21258

2011-03-23 12:58:56,517 | INFO  | Removing Session : SessionInfo {commandId = 71572, responseRequired = false, sessionId = ID:test.customs.am-28506-1300862663044-0:3:15906} | org.apache.activemq.broker.util.LoggingBrokerPlugin | ActiveMQ Transport: ssl:///192.168.0.30:21259

2011-03-23 12:58:56,518 | INFO  | Adding Session : SessionInfo {commandId = 71570, responseRequired = false, sessionId = ID:test.customs.am-28506-1300862663044-0:2:15905} | org.apache.activemq.broker.util.LoggingBrokerPlugin | ActiveMQ Transport: ssl:///192.168.0.30:21258

2011-03-23 12:58:56,518 | INFO  | Removing Session : SessionInfo {commandId = 71570, responseRequired = false, sessionId = ID:test.customs.am-28506-1300862663044-0:2:15905} | org.apache.activemq.broker.util.LoggingBrokerPlugin | ActiveMQ Transport: ssl:///192.168.0.30:21258

2011-03-23 12:58:56,518 | INFO  | Adding Session : SessionInfo {commandId = 71572, responseRequired = false, sessionId = ID:test.customs.am-28506-1300862663044-0:2:15906} | org.apache.activemq.broker.util.LoggingBrokerPlugin | ActiveMQ Transport: ssl:///192.168.0.30:21258

........................

........................

[garytully]

Hi Sargis, thanks, this looks like a nice improvement.

To get it merged to trunk we need a license grant. The easiest way to do that (and the official way) is to attach your patch to a jira at apache and tick the check text box (on the file upload dialog) that indicates that you grant license to ASF for inclusion of your code in ASF works. github is great for forking and sharing, but it bypasses the required license check for a code grant.

see: https://cwiki.apache.org/confluence/display/ACTIVEMQ/Contributing

[armsargis_sargis]

Hi, thanks for reply. Issue created with attached patch: https://issues.apache.org/jira/browse/AMQ-3263.