aries-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Nicolas Dutertry (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ARIES-1885) JpaInterceptor does not properly end coordination in case of Exception
Date Fri, 01 Feb 2019 13:19:00 GMT

    [ https://issues.apache.org/jira/browse/ARIES-1885?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16758286#comment-16758286
] 

Nicolas Dutertry commented on ARIES-1885:
-----------------------------------------

I have created a github project showing the issue : [https://github.com/nicolas-dutertry/test-aries-coordination]

The issue is solved with version 2.7.2-SNAPSHOT.

> JpaInterceptor does not properly end coordination in case of Exception
> ----------------------------------------------------------------------
>
>                 Key: ARIES-1885
>                 URL: https://issues.apache.org/jira/browse/ARIES-1885
>             Project: Aries
>          Issue Type: Bug
>          Components: JPA
>    Affects Versions: jpa-2.7.1
>            Reporter: Nicolas Dutertry
>            Assignee: Christian Schneider
>            Priority: Major
>             Fix For: jpa-2.7.2
>
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> In class JpaInterceptor the method postCallWithException calls Coordination.fail() but
not Coordination.end():
> {code:java}
> @Override
> public void postCallWithException(ComponentMetadata cm, Method m, Throwable ex, Object
preCallToken) {
>     LOG.debug("PostCallWithException for bean {}, method {}", cm.getId(), m.getName(),
ex);
>     if (preCallToken != null) {
>         ((Coordination)preCallToken).fail(ex);
>     }
> }
> {code}
>  
> However, the javadoc of Coordination.fail() states:
> {code:none}
> If this Coordination has been {@link #push() pushed} onto a thread local Coordination
stack, this Coordination is not removed from the stack. The creator of this Coordination must
still call {@link #end()} on this Coordination to cause it to be removed from the thread local
Coordination stack.
> {code}
> This is causing issues for subsequent calls to jpa service because the coordination is
not removed from the stack.
> The method postCallWithException should be implemented like this:
> {code:java}
> @Override
> public void postCallWithException(ComponentMetadata cm, Method m, Throwable ex, Object
preCallToken) {
>     LOG.debug("PostCallWithException for bean {}, method {}", cm.getId(), m.getName(),
ex);
>     if (preCallToken != null) {
>         Coordination coordination = (Coordination)preCallToken;
>         coordination.fail(ex);
>         coordination.end();
>     }
> }
> {code}
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Mime
View raw message