Bug 56920

Summary: LogViewer : Make it receive all log events even when it is closed
Product: JMeter Reporter: UbikLoadPack support <support>
Component: MainAssignee: JMeter issues mailing list <issues>
Status: RESOLVED FIXED    
Severity: enhancement CC: p.mouawad
Priority: P2    
Version: 2.11   
Target Milestone: ---   
Hardware: All   
OS: All   
Attachments: Patch

Description UbikLoadPack support 2014-09-06 13:53:43 UTC
Created attachment 31969 [details]
Patch

Hello,
LogViewer is a nice feature to see easily logs without opening jmeter.log.
Unfortunately, as it is disabled by default, it does not receive log events.
It only does when opened, which requires user to enable it then replay what lead to this error. 

This feature enables LogViewer to receive events even if closed.
It can be controlled by property:
jmeter.loggerpanel.enable_when_closed

But we propose to enable it by default (so changes.xml section needs to mention it) and user can disable it by setting property to false.

The patch we propose also adds a Hand cusor on the Warning Icon to show user it is a clickable element.

As usual, as we signed a CCLA, all rights of this code belong to Apache.

Regards
Ubik Load Pack team
Comment 1 Sebb 2014-09-06 14:18:23 UTC
One reason why it is not enabled by default is that it takes up additional memory. This can be quite considerable if debug is enabled.
Comment 2 UbikLoadPack support 2014-09-06 14:27:08 UTC
Hello,
Yes but there is a built-in protection with property:
jmeter.loggerpanel.maxlength


Being big users of JMeter as long as our customers, we feel it would be great to have this enabled.
It's like in Eclipse, Console receives events even if closed.
Comment 3 Philippe Mouawad 2014-09-07 19:51:57 UTC
Date: Sun Sep  7 19:50:07 2014
New Revision: 1623149

URL: http://svn.apache.org/r1623149
Log:
Bug 56920 - LogViewer : Make it receive all log events even when it is closed
Bugzilla Id: 56920

Modified:
    jmeter/trunk/bin/jmeter.properties
    jmeter/trunk/src/core/org/apache/jmeter/gui/LoggerPanel.java
    jmeter/trunk/src/core/org/apache/jmeter/gui/MainFrame.java
    jmeter/trunk/xdocs/changes.xml
    jmeter/trunk/xdocs/usermanual/hints_and_tips.xml
Comment 4 Philippe Mouawad 2014-09-07 19:52:12 UTC
Thanks for contribution.