Hi Ravi,

My preference would be for you to just check in the CLEAR_TABLES patch without a JIRA, just a good explanation in the commit message. Then there's no question about the CLEAR_TABLES patch being connected somehow to this issue.

The patch to fix the deadlock seems straightforward enough. Are there any performance tests that we can run to see if there is a regression?

Regards,

Craig

On Jul 4, 2009, at 8:58 AM, Ravi P Palacherla (JIRA) wrote:


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

Ravi P Palacherla updated OPENJPA-453:
--------------------------------------

   Attachment: JIRA-453_trunk.patch

Attached patch fixes the deadlock issue on trunk.

It also contains a small change in TestBooleanValue ( includes CLEAR_TABLES in setup). This is not relevant to this problem but it will help pass the test cases.
If a separate JIRA is needed for it, I can open one.

Evicting embedded object nullifies statemanager
-----------------------------------------------

               Key: OPENJPA-453
               URL: https://issues.apache.org/jira/browse/OPENJPA-453
           Project: OpenJPA
        Issue Type: Bug
       Environment: Kodo 4.1.4, Ms sql server 2005, jTDS 1.2, jdk 1.6
          Reporter: Christiaan
          Assignee: Ravi P Palacherla
Attachments: JIRA-453_trunk.patch, OPENJPA-453_trunk_testcase.patch, OpenJPABug453Embedded.zip, openJPATestCase.zip, TestCaseEvictEmbedded.zip


I am noticing the following behaviour: If evict() is called on an embedded object the statemanager is nullified which is in contrast to non- embedded objects. Subsequently, calling JDOHelper.getPersistenceManager() on the
evicted embedded object returns null. Is this the correct behaviour?

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


Craig L Russell
Architect, Sun Java Enterprise System http://db.apache.org/jdo
408 276-5638 mailto:[email protected]
P.S. A good JDO? O, Gasp!

Attachment: smime.p7s
Description: S/MIME cryptographic signature

Reply via email to