openjpa-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Rick Curtis (JIRA)" <j...@apache.org>
Subject [jira] Updated: (OPENJPA-1119) NPE in InstrumentationFactory
Date Wed, 10 Jun 2009 17:23:07 GMT

     [ https://issues.apache.org/jira/browse/OPENJPA-1119?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Rick Curtis updated OPENJPA-1119:
---------------------------------

    Attachment: OPENJPA-1119.PATCH

Refactored code for a more 'complete' fix. This new patch fixes the NPE, but also does some
additional manifest validation. 

If it is determined that the InstrumentationFactory was loaded from a jar file, that jar file
is opened up and the manifest is examined. If the InstrumentationFactory is defined as the
Agent-Class, we will go ahead and use that as an argument to the attach api. If the InstrumentationFactory
isn't defined as the Agent-Class, we will go off and create a temporary file. 

Currently the openjpa-all.xxxx.jar and openjpa-kernal.xx.jar files do not have the InstrumentationFactory
defined as the Agent-Class and I will address that issue in OPENJPA-1117. I can hypothesize
that the InstrumentationFactory could be loaded from a jar that doesn't have the Agent-Class
defined if a user were to repackage OpenJPA and not properly copy the manifest attributes
over.

While testing this patch I uncovered 33 new test failures when running on Sun 1.6. After looking
closer at the failures, most of the tests are named TestUnenhancedxxxxx. After looking at
the commit comment for OPENJPA-293, it appears that it was known that these tests would fail
when using redefinition. Now that it is possible to use redefinition, these tests are failing.
Since it appears [1] that we are going to disable support for the redefinition(subclassing)
I am going to exclude these failing tests from the build. Three other tests are also failing
due to the dynamic enhancer, so I am excluding those tests also.

I also updated the openjpa-persistence-jdbc/pom.xml and added -XX:MaxPermSize=256m to the
test.jvm.arguments to take care of the Sun PermGen OOM problems that I run into when running
on Sun 1.6. 

[1]http://n2.nabble.com/-VOTE--Turn-off-enhancement-by-subclassing-as-the-default-td1616140.html#a2950000

> NPE in InstrumentationFactory
> -----------------------------
>
>                 Key: OPENJPA-1119
>                 URL: https://issues.apache.org/jira/browse/OPENJPA-1119
>             Project: OpenJPA
>          Issue Type: Bug
>          Components: kernel
>    Affects Versions: 2.0.0
>            Reporter: Rick Curtis
>            Assignee: Rick Curtis
>             Fix For: 2.0.0
>
>         Attachments: OPENJPA-1119.PATCH, OPENJPA-1119.patch
>
>
> As reported in the dev forum [1] InstrumentationFactory.class.getProtectionDomain().getCodeSource()
returns null, causing a NPE.
> Caused by: java.lang.NullPointerException
>         at org.apache.openjpa.enhance.InstrumentationFactory.getAgentJar(InstrumentationFactory.java:204)
>         at org.apache.openjpa.enhance.InstrumentationFactory.access$200(InstrumentationFactory.java:47)
>         at org.apache.openjpa.enhance.InstrumentationFactory$1.run(InstrumentationFactory.java:99)
>         at java.security.AccessController.doPrivileged(Native Method)
>         at org.apache.openjpa.enhance.InstrumentationFactory.getInstrumentation(InstrumentationFactory.java:87)
>         at org.apache.openjpa.enhance.PCEnhancerAgent.loadDynamicAgent(PCEnhancerAgent.java:95)
>         at org.apache.openjpa.persistence.PersistenceProviderImpl.loadAgent(PersistenceProviderImpl.java:243)
>         at org.apache.openjpa.persistence.PersistenceProviderImpl.createEntityManagerFactory(PersistenceProviderImpl.java:91)
>         at org.apache.openjpa.persistence.PersistenceProviderImpl.createEntityManagerFactory(PersistenceProviderImpl.java:119)
>         at org.apache.openjpa.persistence.PersistenceProviderImpl.createEntityManagerFactory(PersistenceProviderImpl.java:55)
>         at javax.persistence.Persistence.createFactory(Persistence.java:172)
>         ... 51 more
> [1] http://n2.nabble.com/Error-loading-agent-with-Openjpa2-td3024299.html

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message