Return-Path: Delivered-To: apmail-db-jdo-dev-archive@www.apache.org Received: (qmail 25610 invoked from network); 3 May 2005 20:26:11 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 3 May 2005 20:26:11 -0000 Received: (qmail 64203 invoked by uid 500); 3 May 2005 20:27:56 -0000 Mailing-List: contact jdo-dev-help@db.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: jdo-dev@db.apache.org Delivered-To: mailing list jdo-dev@db.apache.org Received: (qmail 64165 invoked by uid 99); 3 May 2005 20:27:56 -0000 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests= X-Spam-Check-By: apache.org Received-SPF: pass (hermes.apache.org: local policy) Received: from brmea-mail-3.Sun.COM (HELO brmea-mail-3.sun.com) (192.18.98.34) by apache.org (qpsmtpd/0.28) with ESMTP; Tue, 03 May 2005 13:27:55 -0700 Received: from phys-mpk-1 ([129.146.11.81]) by brmea-mail-3.sun.com (8.12.10/8.12.9) with ESMTP id j43KQ7jO027157 for ; Tue, 3 May 2005 14:26:07 -0600 (MDT) Received: from conversion-daemon.mpk-mail1.sfbay.sun.com by mpk-mail1.sfbay.sun.com (iPlanet Messaging Server 5.2 HotFix 1.24 (built Dec 19 2003)) id <0IFX00C01KBHKS@mpk-mail1.sfbay.sun.com> (original mail from Michelle.Caisse@Sun.COM) for jdo-dev@db.apache.org; Tue, 03 May 2005 13:26:07 -0700 (PDT) Received: from [129.150.26.11] (vpn-129-150-26-11.SFBay.Sun.COM [129.150.26.11]) by mpk-mail1.sfbay.sun.com (iPlanet Messaging Server 5.2 HotFix 1.24 (built Dec 19 2003)) with ESMTP id <0IFX00E99KQV9A@mpk-mail1.sfbay.sun.com> for jdo-dev@db.apache.org; Tue, 03 May 2005 13:25:43 -0700 (PDT) Date: Tue, 03 May 2005 13:29:30 -0700 From: Michelle Caisse Subject: Re: pmf not closed In-reply-to: <4277DC9F.9000802@spree.de> To: jdo-dev@db.apache.org Message-id: <4277DF2A.3020400@sun.com> MIME-version: 1.0 Content-type: text/plain; charset=ISO-8859-1; format=flowed Content-transfer-encoding: 7BIT X-Accept-Language: en-us, en User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.7.2) Gecko/20040803 References: <42727265.7070100@sun.com> <42753BBE.6060000@spree.de> <42768B2B.70600@sun.com> <4276A522.5030104@spree.de> <4276BA65.5000700@sun.com> <4277DC9F.9000802@spree.de> X-Virus-Checked: Checked X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N Hi, Michael, Responses in-line... Michael Bouschen wrote: > Hi Michelle, > > thanks for testing. > >> Hi, Michael, >> >> The tests still don't pass because you are attempting to delete the >> PCPoints before the PCRects. Even though it is one transaction, you >> have to change the order as I did below. > > > I agree we need to change the order and delete PCRect instances first > as you suggest. The cleanup code should succeed in any case. But maybe > this scenario is an interesting test for the TCK delete test. I think > the JDO implementation should reorder the SQL DELETE statements > according to the foreign keys in the database such that the > transaction succeeds no matter which instances the user deleted first > in the JDO transaction. Interesting point. I hadn't considered that. Is it in the spec? > >> >> The cleanup appears to happen correctly now so the PMF is closed. >> But I don't see a failure message, just "FAIL". I don't know why >> this is. > > > Yes, just getting "FAIL" is strange. I need to run the test cases you > suggest below and figure out the details. In general, the tests are not giving as much feedback when they fail as I would expect. > > Does this hold you up or does the current changes of JDO_Test and > PersistenceManagerTest allow you to continue with the TCK tests? This is not holding me up. You can check in now or later, whatever works for you. Thanks!! -- Michelle > > Regards Michael > >> >> I am using CurrentTransaction as the test case. I believe that >> ConcurrentPersistenceManagersSameClasses leaves behind the PCRects >> that PersistenceManagerTest then tries to delete when >> CurrentTransaction is run. These two run in sequence when you run >> maven runtck.jdori, so you can just watch for them to scroll by. Or >> you can be more clever and just run the two tests in sequence. >> >> -- Michelle >