batchee-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hudson (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (BATCHEE-56) ChunkListener.onError() should not be called in case of exceptions outside of chunks
Date Tue, 09 Dec 2014 11:38:12 GMT

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

Hudson commented on BATCHEE-56:
-------------------------------

SUCCESS: Integrated in batchee #547 (See [https://builds.apache.org/job/batchee/547/])
BATCHEE-56 reader/writer close shouldn't call onError as well (rmannibucau: rev bcb8dbe81a40b977bdaf37af3483b910f37fcffb)
* jbatch/src/main/java/org/apache/batchee/container/impl/controller/chunk/ChunkStepController.java


> ChunkListener.onError() should not be called in case of exceptions outside of chunks
> ------------------------------------------------------------------------------------
>
>                 Key: BATCHEE-56
>                 URL: https://issues.apache.org/jira/browse/BATCHEE-56
>             Project: BatchEE
>          Issue Type: Bug
>          Components: jbatch-core
>    Affects Versions: 0.1-incubating, 0.2-incubating
>            Reporter: Mario Ködding
>            Assignee: Romain Manni-Bucau
>            Priority: Minor
>             Fix For: 0.3-incubating
>
>
> As we have some integration-tests running on different application-servers, we discovered
some differences in the behavior of the jbatch-implementations concerning listeners in case
of exceptions:
> If an exception occurs inside the ItemReader/ItemWriter open() or close() methods, then
in case of JBeret, the ChunkListener.onError() method is not called, while the JBatch RI included
in GlassFish 4 as well as Apache BatchEE do call the method.
> This issue has already been discussed in the JBeret issue-tracker:
> https://github.com/jberet/jsr352/issues/25
> Cheng Fang summarizes it as follows: 
> "A chunk-type step consists of multiple chunks, and ChunkListener is for these smaller
chunks, not for the entire step. When ItemReader.open() is called at the beginning of the
step, we haven't come to any chunk yet, why should the ChunkListener be called?"
> I agree with him, that according to the spec, the ChunkListener should not be called
in case of exceptions inside the open() and close() methods. So I think the BatchEE implementation
should be adjusted here. Please refer to the JBeret issue linked above for more details.
> Related to this issue, I have opened a discussion on the jbatch public mailing list for
enhancing the error-handling with listeners:
> https://java.net/projects/jbatch/lists/public/archive/2014-12/message/3



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

Mime
View raw message