felix-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Pierre De Rop (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (FELIX-4602) TemporalServiceDependency does not properly propagate RuntimeExceptions
Date Thu, 07 Aug 2014 08:44:12 GMT

    [ https://issues.apache.org/jira/browse/FELIX-4602?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14089007#comment-14089007

Pierre De Rop commented on FELIX-4602:

Thanks Sean for the proposed patch.

As explained in FELIX-4598, we have made a big revamp of dependency manager, which is now
using a new (almost) lock-free thread model. 

This new DM 4.0.0 version is unfortunately not yet committed in felix-trunk, but in the felix
sandbox: [1] , and the code has changed a lot from DM 3.2.0; it's a big refactoring, so I
would to avoid fixing anything in DM 3.2.0 because we first have to bring this new DM 4.0.0
version from the sandbox into the felix-trunk, then at this point I will will apply your patch
(thanks again !).

(I'm currently off and will get back from vacations around 20 august).

[1] http://svn.apache.org/viewvc/felix/sandbox/pderop/dependencymanager-prototype/

> TemporalServiceDependency does not properly propagate RuntimeExceptions
> -----------------------------------------------------------------------
>                 Key: FELIX-4602
>                 URL: https://issues.apache.org/jira/browse/FELIX-4602
>             Project: Felix
>          Issue Type: Bug
>          Components: Dependency Manager
>    Affects Versions: dependencymanager-3.2.0
>            Reporter: Sean Rohead
>            Assignee: Pierre De Rop
>              Labels: easyfix
>             Fix For: dependencymanager-4.0.0
>         Attachments: TemporalServiceDependencyImpl.java.patch
> I have a service that throws subclasses of RuntimeException from some of it's methods.
When wrapped in a TemporalServiceDependency, these calls throw UndeclaredThrowableException
> The fix is quite simple:
> Wrap TemporalServiceDependencyImpl lines 182-188 in the following try/catch block:
> try {
>     ...
> }
> catch (InvocationTargetException e) {
>     throw e.getTargetException();
> }

This message was sent by Atlassian JIRA

View raw message