Bug 900690 (JBPAPP6-1413) - CLONE - HHH-7435 Performance bottleneck on Javassist
Summary: CLONE - HHH-7435 Performance bottleneck on Javassist
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: JBPAPP6-1413
Product: JBoss Enterprise Application Platform 6
Classification: JBoss
Component: JPA
Version: 6.0.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ER1
: EAP 6.2.0
Assignee: Scott Marlow
QA Contact:
URL: http://jira.jboss.org/jira/browse/JBP...
Whiteboard:
: 977376 (view as bug list)
Depends On:
Blocks: 1004020
TreeView+ depends on / blocked
 
Reported: 2012-07-05 15:11 UTC by Alessandro Lazarotti
Modified: 2018-12-03 18:09 UTC (History)
9 users (show)

Fixed In Version:
Clone Of:
Environment:
Sun/Oracle JDK 1.6_32, Linux 64b
Last Closed: 2013-12-15 16:13:03 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker JASSIST-163 0 Major Closed RuntimeSupport.find2Methods a perf hotspot when proxy's methods are called at higher concurrency 2013-11-11 13:57:47 UTC
Red Hat Issue Tracker JBPAPP6-1413 0 Major Closed CLONE - HHH-7435 Performance bottleneck on Javassist 2013-11-11 13:57:47 UTC

Description Alessandro Lazarotti 2012-07-05 15:11:32 UTC
Help Desk Ticket Reference: https://c.na7.visual.force.com/apex/Case_View?id=500A000000A4nT3IAJ&sfdc.override=1
project_key: JBPAPP6

The method javassist.util.proxy.RuntimeSupport.find2methods has a synch block, called indirectly by ProxyFactory.createClass(). It is a strong bottleneck for frameworks as Hibernate which can use it several times in a large set of data.

More details in https://issues.jboss.org/browse/JASSIST-163

Comment 1 Alessandro Lazarotti 2012-07-05 15:11:33 UTC
Link: Added: This issue Cloned from JBPAPP-9454


Comment 2 Alessandro Lazarotti 2012-07-05 15:11:33 UTC
Link: Added: This issue relates to JASSIST-163


Comment 3 Alessandro Lazarotti 2012-07-05 15:12:24 UTC
Labels: Removed: gss_required 


Comment 4 Alessandro Lazarotti 2012-07-05 21:23:34 UTC
Link: Added: This issue is related to AS7-5127


Comment 5 Scott Marlow 2012-07-10 18:05:56 UTC
The https://issues.jboss.org/browse/JASSIST-163 fix needs more work.  See discussion on JASSIST-163 & http://lists.jboss.org/pipermail/jboss-as7-dev/2012-July/006274.html



Comment 6 Scott Marlow 2012-07-31 18:43:12 UTC
We don't have a javassist change yet that is safe to bring in.

Comment 7 Scott Marlow 2012-07-31 19:10:06 UTC
https://hibernate.onjira.com/browse/HHH-5684 is related.

Comment 8 Brett Meyer 2012-09-14 17:24:48 UTC
On hold until Javassist 3.17.0.GA released

Comment 9 Scott Marlow 2012-10-26 15:37:03 UTC
Other AS modules that depend on javassist (via static module dependencies) are Weld, jbossws-native-core, scannotation.

Comment 10 Anne-Louise Tangring 2012-11-13 20:57:56 UTC
Docs QE Status: Removed: NEW 


Comment 11 Brett Meyer 2013-01-03 19:38:06 UTC
Hibernate 4.1.9.Final stepped up to Javassist 3.17.1.  All other EAP components, dependent on Javassist, tested and verified.  EAP 6.1.0 will be based at least on 4.1.9, if not later.  Marking this as resolved.

Comment 12 Brett Meyer 2013-01-10 16:57:33 UTC
Reopening.  Javassist 3.17.1 creates a regression:

https://hibernate.onjira.com/browse/HHH-7884
https://issues.jboss.org/browse/JASSIST-189

Our current spec for EAP 6.1.0 is using Javassist 3.15.0-GA-redhat-2 until this is corrected.

Comment 13 Scott Marlow 2013-01-11 14:49:16 UTC
Due to the Javassist regression with "bridge methods", I think its likely that we will retarget this issue to the next EAP 6.x release.  

Comment 15 Brett Meyer 2013-06-18 16:57:44 UTC
Javassist 3.18.0.GA is released and will be updated in Hibernate upstream in https://hibernate.atlassian.net/browse/HHH-8288.  We'll push into 6.2.0 after it's been through a few upstream releases.  Scott and I would be more confident if it went through a Wildfly release as well.

Comment 16 Brett Meyer 2013-06-27 03:44:33 UTC
*** Bug 977376 has been marked as a duplicate of this bug. ***

Comment 17 Dimitris Andreadis 2013-08-27 12:45:00 UTC
Propose for EAP 6.2

Comment 18 Brett Meyer 2013-08-27 14:11:06 UTC
Moving to JPA component -- this needs upgraded in EAP itself.

Comment 19 Paul Gier 2013-09-03 17:47:20 UTC
Changing state to MODIFIED because the issue has been fixed in the component (Javassist).  Created bug 1004020 to track merging the upgrade into EAP sources.

Comment 20 Zbyněk Roubalík 2013-09-23 11:19:04 UTC
EAP 6.2.0.ER1


Note You need to log in before you can comment on or make changes to this bug.