Mailing-List: contact dev-help@jackrabbit.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@jackrabbit.apache.org
Message-ID: <1277502112.1236092456599.JavaMail.jira@brutus>
Date: Tue, 3 Mar 2009 07:00:56 -0800 (PST)
From: =?utf-8?Q?Claus_K=C3=B6ll_=28JIRA=29?= <jira@apache.org>
To: dev@jackrabbit.apache.org
Subject: [jira] Commented: (JCR-1633) When node is created and locked in
 same transaction, exception is thrown
In-Reply-To: <1262400537.1212161805981.JavaMail.jira@brutus>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable


    [ https://issues.apache.org/jira/browse/JCR-1633?page=3Dcom.atlassian.j=
ira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=3D126783=
37#action_12678337 ]=20

Claus K=C3=B6ll commented on JCR-1633:
---------------------------------

As Alexander wrote i give him partly right.  I understand the specification=
 in that way that you can not lock a node till it has pending changes.
In the context of a transaction it is a little bit difference. I see the Tr=
ansaction as a atomic operation so you should be able to lock a node after =
create and save in one transaction
although the persistence moves to the commit of the transaction.

I will provide a patch that handles this behaviour with some more tescases =
so we can see if it works as described in the specification.

> When node is created and locked in same transaction, exception is thrown
> ------------------------------------------------------------------------
>
>                 Key: JCR-1633
>                 URL: https://issues.apache.org/jira/browse/JCR-1633
>             Project: Jackrabbit Content Repository
>          Issue Type: Bug
>          Components: jackrabbit-core, locks, transactions
>    Affects Versions: core 1.4.3, core 1.4.4
>         Environment: Jackrabbit Core 1.4.4, Jencks 2.0, Springmodules 0.8=
a, Jackrabbit JCA 1.4
>            Reporter: Roman Puchkovskiy
>         Attachments: test-create-and-lock-in-same-tx.zip, XALockTest.java
>
>
> Following code fails when executed inside an XA transaction:
> Node n =3D session.getRootNode().addNode("n");
> n.addMixin("mix:lockable");
> session.save();
> Lock lock =3D n.lock(false, false);
> Stacktrace is
> Caused by: javax.transaction.xa.XAException
> =09at org.apache.jackrabbit.core.TransactionContext.prepare(TransactionCo=
ntext.java:155)
> =09at org.apache.jackrabbit.core.XASessionImpl.commit(XASessionImpl.java:=
337)
> =09at org.apache.jackrabbit.jca.TransactionBoundXAResource.commit(Transac=
tionBoundXAResource.java:39)
> =09at org.apache.geronimo.transaction.manager.WrapperNamedXAResource.comm=
it(WrapperNamedXAResource.java:47)
> =09at org.apache.geronimo.transaction.manager.TransactionImpl.commit(Tran=
sactionImpl.java:301)
> =09... 32 more
> Caused by: org.apache.jackrabbit.core.TransactionException: Unable to upd=
ate.
> =09at org.apache.jackrabbit.core.lock.XAEnvironment.prepare(XAEnvironment=
.java:275)
> =09at org.apache.jackrabbit.core.lock.XALockManager.prepare(XALockManager=
.java:245)
> =09at org.apache.jackrabbit.core.TransactionContext.prepare(TransactionCo=
ntext.java:138)
> =09... 36 more
> Caused by: javax.jcr.ItemNotFoundException: failed to build path of 48fb5=
9d8-ac77-4b9f-8b53-9f2492dca5e5: 48fb59d8-ac77-4b9f-8b53-9f2492dca5e5: 48fb=
59d8-ac77-4b9f-8b53-9f2492dca5e5
> =09at org.apache.jackrabbit.core.HierarchyManagerImpl.getPath(HierarchyMa=
nagerImpl.java:407)
> =09at org.apache.jackrabbit.core.CachingHierarchyManager.getPath(CachingH=
ierarchyManager.java:272)
> =09at org.apache.jackrabbit.core.lock.LockManagerImpl.getPath(LockManager=
Impl.java:651)
> =09at org.apache.jackrabbit.core.lock.LockManagerImpl.internalLock(LockMa=
nagerImpl.java:276)
> =09at org.apache.jackrabbit.core.lock.XAEnvironment$LockInfo.update(XAEnv=
ironment.java:409)
> =09at org.apache.jackrabbit.core.lock.XAEnvironment.prepare(XAEnvironment=
.java:273)
> =09... 38 more
> Caused by: org.apache.jackrabbit.core.state.NoSuchItemStateException: 48f=
b59d8-ac77-4b9f-8b53-9f2492dca5e5
> =09at org.apache.jackrabbit.core.state.SessionItemStateManager.getItemSta=
te(SessionItemStateManager.java:189)
> =09at org.apache.jackrabbit.core.HierarchyManagerImpl.getItemState(Hierar=
chyManagerImpl.java:188)
> =09at org.apache.jackrabbit.core.HierarchyManagerImpl.getPath(HierarchyMa=
nagerImpl.java:402)
> =09... 43 more

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