Issue Details (XML | Word | Printable)

Key: BATCH-216
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Lucas Ward
Reporter: Robert Kasanicky
Votes: 1
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Spring Batch

ItemWriterItemProcessor#doProcess (..) and ItemTransformer#transform(..) should declare "throws Exception"

Created: 26/Nov/07 06:29 AM   Updated: 07/Aug/08 10:06 AM
Component/s: Infrastructure
Affects Version/s: 1.0-m3
Fix Version/s: 1.0-m3
Security Level: Public (Public Issues)

Time Tracking:
Original Estimate: 0.06d
Original Estimate - 0.06d
Remaining Estimate: 0d
Time Spent - 0.12d
Time Spent: 0.12d
Time Spent - 0.12d

File Attachments: 1. Text File processingException.patch (3 kB)



 Description  « Hide
ItemProcessor#process(..) throws exception, so the ItemWriterItemProcessor#doProcess (..) should declare that too.

I think the same makes sense for the ItemTransformer interface (which is assumed to be used as part of processing). Otherwise implementations tend to be polluted with a lot of dumb exception rethrowing (e.g. when the item is a File object).

 All   Comments   Work Log   Change History   FishEye   Builds      Sort Order: Ascending order - Click to sort in descending order
Lucas Ward added a comment - 04/Dec/07 01:31 PM
All ItemProcessor implementations now have throws Exception on abstract methods.

Dave Syer added a comment - 07/Aug/08 10:06 AM
Assume closed as resolved and released