Hi Armin,

I just did a cvs diff on the source and this is what I got:

RCS file:
/home/cvspublic/jakarta-ojb/src/java/org/apache/ojb/odmg/ObjectEnvelopeTable
..jav
a,v
retrieving revision 1.18
diff -r1.18 ObjectEnvelopeTable.java
63a64
> import java.util.Vector;
215c216,219
<             iter = mvOrderOfIds.iterator();
---
> Vector tmp = new Vector();
> tmp.addAll( mvOrderOfIds );
>             //iter = mvOrderOfIds.iterator();
> iter = tmp.iterator();

It seems that the source is the same as what I used when the problem came
up. Are you sure that the changed file is ObjectEnvelopeTable.java. Is my
revision right?

--Bill.


----- Original Message -----
From: "Armin Waibel" <[EMAIL PROTECTED]>
To: "OJB Users List" <[EMAIL PROTECTED]>
Sent: Friday, March 07, 2003 12:55 AM
Subject: Re: ODMG recursive locking / ConcurrentModificationException


> Hi Bill,
>
> checked in that fix? yesterday after reading your post ;-)
> Would be nice if you could try out my solution
> (changed: ObjectEnvelopeTable.java).
>
> regards,
> Armin
>
> ----- Original Message -----
> From: "V.B. Skrypnyk" <[EMAIL PROTECTED]>
> To: "OJB Users List" <[EMAIL PROTECTED]>
> Sent: Friday, March 07, 2003 8:21 AM
> Subject: Re: ODMG recursive locking / ConcurrentModificationException
>
>
> > Hi Armin,
> >
> > I modified the code manually by adding the temp vector solution. That
> > worked.
> > If your fix w/o temp vector was checked in the CVS before 2-3 days ago
> > (that's when I encountered it in the newly downloaded CVS version),
> > then the problem still persists.
> >
> > Cheers,
> > --Bill.
> >
> > ----- Original Message -----
> > From: "Armin Waibel" <[EMAIL PROTECTED]>
> > To: "OJB Users List" <[EMAIL PROTECTED]>
> > Sent: Thursday, March 06, 2003 12:15 AM
> > Subject: Re: ODMG recursive locking / ConcurrentModificationException
> >
> >
> > > Hi Bill,
> > >
> > > I checked in a fix? in CVS without using a temp vector.
> > > Did this solution works for you?
> > >
> > > regards,
> > > Armin
> > >
> > > ----- Original Message -----
> > > From: "V.B. Skrypnyk" <[EMAIL PROTECTED]>
> > > To: "OJB Users List" <[EMAIL PROTECTED]>
> > > Sent: Thursday, March 06, 2003 7:32 AM
> > > Subject: Re: ODMG recursive locking /
> ConcurrentModificationException
> > >
> > >
> > > > I just added code for a temporary vector and it worked. The change
> > > below
> > > > must have gotten lost.
> > > > Or was there a problem with the solution?
> > > > --Bill.
> > > >
> > > > ----- Original Message -----
> > > > From: "V.B. Skrypnyk" <[EMAIL PROTECTED]>
> > > > To: "OJB Users List" <[EMAIL PROTECTED]>
> > > > Sent: Wednesday, March 05, 2003 10:24 PM
> > > > Subject: ODMG recursive locking / ConcurrentModificationException
> > > >
> > > >
> > > > > Hi,
> > > > >
> > > > > I found a thread on the list
> > > > >
> > > >
> > >
> (http://archives.apache.org/eyebrowse/[EMAIL PROTECTED]
> > > apac
> > > > > he.org&msgId=593039) in which it is said:
> > > > > >Indeed you stumbled across a bug in the ObjectEnvelopeTable
> class.
> > > The
> > > > > >bug was fixed on 2002/12/23 13:27:59 which means it is not
> > > available on
> > > > > >version 0.9.8. It is however available directly from the cvs
> > > server.
> > > > > >
> > > > > >The Concurrent Modification Exception occurs while upgrading
> the
> > > > > >locks in a commit operation of the ObjectEnvelopeTable class.
> The
> > > > > >solution we came up with was to iterate over temporary vector
> while
> > > > > >upgrading the locks.
> > > > > >
> > > > > >Cheers,
> > > > > >Luis Cruz
> > > > > I have just downloaded the latest cvs version, and I have the
> same
> > > > problem.
> > > > > Did the change get lost by any chance?
> > > > > --Bill
> > > > > My stack trace:
> > > > >      [java] java.util.ConcurrentModificationException     [java]
> > > at
> > > > >
> > >
> java.util.AbstractList$Itr.checkForComodification(AbstractList.java:444)
> > > > > [java] [org.apache.ojb.odmg.ObjectEnvelopeTable] ERROR: Commit
> on
> > > object
> > > > > level failed for tx [EMAIL PROTECTED]
> > > [java]
> > > > > at java.util.AbstractList$Itr.next(AbstractList.java:417)
> [java]
> > > > at
> > > > >
> > > >
> > >
> org.apache.ojb.odmg.ObjectEnvelopeTable.commitAllEnvelopes(ObjectEnvelop
> > > eTab
> > > > > le.java:218)     [java]     at
> > > > >
> > > >
> > >
> org.apache.ojb.odmg.ObjectEnvelopeTable.commit(ObjectEnvelopeTable.java:
> > > 187)
> > > > > [java] null     [java] [org.apache.ojb.odmg.TransactionImpl]
> ERROR:
> > > Could
> > > > > not prepare for commit: null     [java]     at
> > > > >
> > > >
> > >
> org.apache.ojb.odmg.TransactionImpl.doCommitOnObjects(TransactionImpl.ja
> > > va:3
> > > > > 26)     [java]     at
> > > > >
> > >
> org.apache.ojb.odmg.TransactionImpl.prepare(TransactionImpl.java:575)
> > > > > [java]     at
> > > > >
> org.apache.ojb.odmg.TransactionImpl.commit(TransactionImpl.java:525)
> > > > > [java]     at test.Test.main(Test.java:70)     [java]
> > > > > [org.apache.ojb.broker.accesslayer.ConnectionManagerImpl] INFO:
> > > Rollback
> > > > was
> > > > > called, do rollback on current connection
> > > > > [EMAIL PROTECTED]     [java]
> > > > > java.sql.SQLException: General error: Warning:  Some
> > > non-transactional
> > > > > changed tables couldn't be rolled back     [java]     at
> > > > > org.gjt.mm.mysql.MysqlIO.sendCommand(MysqlIO.java:497)
> [java]
> > > > > [org.apache.ojb.broker.accesslayer.ConnectionManagerImpl] ERROR:
> > > Rollback
> > > > on
> > > > > the underlying connection failed     [java]     at
> > > > > org.gjt.mm.mysql.MysqlIO.sqlQueryDirect(MysqlIO.java:550)
> [java]
> > > > at
> > > > > org.gjt.mm.mysql.MysqlIO.sqlQuery(MysqlIO.java:635)     [java]
> > > General
> > > > > error: Warning:  Some non-transactional changed tables couldn't
> be
> > > rolled
> > > > > back     [java]     at
> > > > > org.gjt.mm.mysql.Connection.execSQL(Connection.java:882)
> [java]
> > > at
> > > > > org.gjt.mm.mysql.Connection.execSQL(Connection.java:815)
> [java]
> > > at
> > > > > org.gjt.mm.mysql.Connection.rollback(Connection.java:551)
> [java]
> > > > at
> > > > >
> > > >
> > >
> com.p6spy.engine.logging.P6LogConnection.rollback(P6LogConnection.java:1
> > > 45)
> > > > > [java]     at
> > > > >
> > > >
> > >
> org.apache.ojb.broker.accesslayer.ConnectionManagerImpl.localRollback(Co
> > > nnec
> > > > > tionManagerImpl.java:245)     [java]     at
> > > > >
> > > >
> > >
> org.apache.ojb.broker.singlevm.PersistenceBrokerImpl.abortTransaction(Pe
> > > rsis
> > > > > tenceBrokerImpl.java:358)     [java]     at
> > > > >
> > > >
> > >
> org.apache.ojb.broker.singlevm.PersistenceBrokerImpl.close(PersistenceBr
> > > oker
> > > > > Impl.java:320)     [java]     at
> > > > >
> > > >
> > >
> org.apache.ojb.broker.singlevm.DelegatingPersistenceBroker.close(Delegat
> > > ingP
> > > > > ersistenceBroker.java:132)     [java]     at
> > > > >
> > > >
> > >
> org.apache.ojb.broker.singlevm.PoolablePersistenceBroker.close(PoolableP
> > > ersi
> > > > > stenceBroker.java:28)     [java]     at
> > > > >
> > >
> org.apache.ojb.odmg.TransactionImpl.doClose(TransactionImpl.java:424)
> > > > > [java]     at
> > > > >
> org.apache.ojb.odmg.TransactionImpl.commit(TransactionImpl.java:548)
> > > > > [java]     at test.Test.main(Test.java:70)     [java]
> > > > > org.odmg.TransactionAbortedException     [java]     at
> > > > >
> > > >
> > >
> org.apache.ojb.odmg.ObjectEnvelopeTable.commit(ObjectEnvelopeTable.java:
> > > 198)
> > > > > [java]     at
> > > > >
> > > >
> > >
> org.apache.ojb.odmg.TransactionImpl.doCommitOnObjects(TransactionImpl.ja
> > > va:3
> > > > > 26)     [java]     at
> > > > >
> > >
> org.apache.ojb.odmg.TransactionImpl.prepare(TransactionImpl.java:575)
> > > > > [java]     at
> > > > >
> org.apache.ojb.odmg.TransactionImpl.commit(TransactionImpl.java:525)
> > > > > [java]     at test.Test.main(Test.java:70)     [java] Exception
> in
> > > thread
> > > > > "main"     [java] Java Result: 1
> > > > >
> > > > >
> > > > >
> > > >
> > >
> > ---------------------------------------------------------------------
> > > > > To unsubscribe, e-mail: [EMAIL PROTECTED]
> > > > > For additional commands, e-mail: [EMAIL PROTECTED]
> > > > >
> > > > >
> > > >
> > > >
> > >
> > ---------------------------------------------------------------------
> > > > To unsubscribe, e-mail: [EMAIL PROTECTED]
> > > > For additional commands, e-mail: [EMAIL PROTECTED]
> > > >
> > > >
> > > >
> > >
> > >
> >
> > ---------------------------------------------------------------------
> > > To unsubscribe, e-mail: [EMAIL PROTECTED]
> > > For additional commands, e-mail: [EMAIL PROTECTED]
> > >
> > >
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: [EMAIL PROTECTED]
> > For additional commands, e-mail: [EMAIL PROTECTED]
> >
> >
> >
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to