batchee-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Romain Manni-Bucau (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (BATCHEE-65) remove unnecessary try/catch in CDIBatchArtifactFactory
Date Tue, 26 May 2015 14:21:20 GMT

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

Romain Manni-Bucau commented on BATCHEE-65:
-------------------------------------------

Great, if you need any storage maybe we should introduce a CDIService. This would inherit
from ServicesManager which is already handled correctly either as singleton or by app. Then
this service would do the detection or store the bm.

> remove unnecessary try/catch in CDIBatchArtifactFactory
> -------------------------------------------------------
>
>                 Key: BATCHEE-65
>                 URL: https://issues.apache.org/jira/browse/BATCHEE-65
>             Project: BatchEE
>          Issue Type: Improvement
>          Components: jbatch-core
>    Affects Versions: 0.3-incubating
>            Reporter: Reinhard Sandtner
>            Priority: Minor
>             Fix For: 0.3-incubating
>
>         Attachments: BATCHEE-65.patch
>
>
> resolving a CDI-Bean is in a try/catch block which catches an Exception and does nothing.
this hides useful information if an Exception has been thrown.
> you get a NPE if ie the reader could not be resolved but not the real exception.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message