[Bug 56920] New: LogViewer : Make it receive all log events even when it is closed

[bu...@apache.org]

https://issues.apache.org/bugzilla/show_bug.cgi?id=56920

            Bug ID: 56920
           Summary: LogViewer : Make it receive all log events even when
                    it is closed
           Product: JMeter
           Version: 2.11
          Hardware: All
                OS: All
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: Main
          Assignee: issues@jmeter.apache.org
          Reporter: support@ubikloadpack.com

Created attachment 31969
  --> https://issues.apache.org/bugzilla/attachment.cgi?id=31969&action=edit
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

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 56920] LogViewer : Make it receive all log events even when it is closed

[bu...@apache.org]

https://issues.apache.org/bugzilla/show_bug.cgi?id=56920

--- Comment #4 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
Thanks for contribution.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 56920] LogViewer : Make it receive all log events even when it is closed

[bu...@apache.org]

https://issues.apache.org/bugzilla/show_bug.cgi?id=56920

--- Comment #2 from UbikLoadPack support <su...@ubikloadpack.com> ---
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.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 56920] LogViewer : Make it receive all log events even when it is closed

[bu...@apache.org]

https://issues.apache.org/bugzilla/show_bug.cgi?id=56920

Philippe Mouawad <p....@ubik-ingenierie.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #31969|0                           |1
           is patch|                            |
                 CC|                            |p.mouawad@ubik-ingenierie.c
                   |                            |om

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 56920] LogViewer : Make it receive all log events even when it is closed

[bu...@apache.org]

https://issues.apache.org/bugzilla/show_bug.cgi?id=56920

Philippe Mouawad <p....@ubik-ingenierie.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #3 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
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

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 56920] LogViewer : Make it receive all log events even when it is closed

[bu...@apache.org]

https://issues.apache.org/bugzilla/show_bug.cgi?id=56920

--- Comment #1 from Sebb <se...@apache.org> ---
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.

-- 
You are receiving this mail because:
You are the assignee for the bug.