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 Fri, 22 May 2015 11:11:17 GMT

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

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

was the question, main point is depending how you patch it I think it is better to have a
single commit to rework the CDI integration. If you feel it not adapted another jira is fine.

> 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