You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@jmeter.apache.org by bu...@apache.org on 2013/12/08 15:52:04 UTC

[Bug 55858] New: Startup Performance : On Startup, BeanInfoSupport should test for key availability instead of throwing

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

            Bug ID: 55858
           Summary: Startup Performance : On Startup, BeanInfoSupport
                    should test for key availability instead of throwing
           Product: JMeter
           Version: 2.6
          Hardware: PC
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Main
          Assignee: issues@jmeter.apache.org
          Reporter: p.mouawad@ubik-ingenierie.com

At startup BeanInfoSupport constructor fetches i18n values catching
MissingResourceException.
Since java 6, it is better to test for key existence

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

[Bug 55858] Startup Performance : On Startup, BeanInfoSupport should test for key availability instead of throwing

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=55858

--- Comment #4 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
40 times.

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

[Bug 55858] Startup Performance : On Startup, BeanInfoSupport should test for key availability instead of throwing

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=55858

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

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

--- Comment #6 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
Closing as no more comment.
Feel free to reopen if you want me to change

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

[Bug 55858] Startup Performance : On Startup, BeanInfoSupport should test for key availability instead of throwing

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=55858

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

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

--- Comment #2 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
Date: Sun Dec  8 14:53:49 2013
New Revision: 1549033

URL: http://svn.apache.org/r1549033
Log:
Bug 55858 - Startup Performance : On Startup, BeanInfoSupport should test for
key availability instead of throwing
Bugzilla Id: 55858

Modified:
    jmeter/trunk/src/core/org/apache/jmeter/testbeans/BeanInfoSupport.java
    jmeter/trunk/xdocs/changes.xml

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

[Bug 55858] Startup Performance : On Startup, BeanInfoSupport should test for key availability instead of throwing

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=55858

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |p.mouawad@ubik-ingenierie.c
                   |                            |om
           Hardware|PC                          |All
                 OS|                            |All
           Severity|normal                      |enhancement

--- Comment #1 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
Affects all versions up to 2.10

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

[Bug 55858] Startup Performance : On Startup, BeanInfoSupport should test for key availability instead of throwing

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=55858

Sebb <se...@apache.org> changed:

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

--- Comment #3 from Sebb <se...@apache.org> ---
Is the exception frequently generated?

If not, checking to ensure it does not happen is likely a waste of time. Also
the new code creates the string twice; this is unnecessary and somewhat fragile
- the same field should be used for both to enure they are identical.

If this is a frequent code pattern, there should probably be a common helper
method to do the check.

This is quite different from Bug 55857 where an empty string does occur
frequently.

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

[Bug 55858] Startup Performance : On Startup, BeanInfoSupport should test for key availability instead of throwing

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=55858

--- Comment #5 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
Date: Sun Dec  8 20:10:53 2013
New Revision: 1549201

URL: http://svn.apache.org/r1549201
Log:
Bug 55858 - Startup Performance : On Startup, BeanInfoSupport should test for
key availability instead of throwing
Avoid duplicate string creation
Bugzilla Id: 55858

Modified:
    jmeter/trunk/src/core/org/apache/jmeter/testbeans/BeanInfoSupport.java

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