History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: BATCH-518
Type: Improvement Improvement
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Robert Kasanicky
Reporter: Robert Kasanicky
Votes: 0
Watchers: 0
Operations

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

clean up the *Or* repository methods

Created: 27/Mar/08 08:27 AM   Updated: 30/Jul/08 04:41 AM
Component/s: Core
Affects Version/s: 1.0.0
Fix Version/s: 2.0.0.M1

Time Tracking:
Issue & Sub-Tasks
Issue Only
Original Estimate: 1.5d
Original Estimate - 1.5d
Remaining Estimate: 0.5d
Time Spent - 1.62d Remaining Estimate - 0.5d
Time Spent: 1.62d
Time Spent - 1.62d Remaining Estimate - 0.5d

Issue Links:
Depends
 

Sub-Tasks  All   Open   

 Description  « Hide
Repository defaults to using saveOrUpdate methods, which unnecessarily blurs distinction between save and update.

1) saveOrUpdate(JobExecution) can be simply update(JobExecution), creation (and save) is handled by createJobExecution(..)
2) saveOrUpdate(StepExecution) - saved before processing records, updated in the processing loop => easy to split + shouldn't repository have createStepExecution(JobInstance, Step) similar to createJobExecution(Job, JobParameters) to be consistent?
3) saveOrUpdateExecutionContext(StepExecution) - same as 2)

 All   Comments   Work Log   Change History   FishEye   Related Builds      Sort Order: Ascending order - Click to sort in descending order
Robert Kasanicky - 28/Apr/08 05:33 AM
Moving to 2.0 as the change would cause API incompatibility (although most users are unlikely to use this interface directly).

Robert Kasanicky - 28/Jul/08 03:30 AM
rescheduled to M1, as the dependent BATCH-654 is already M1

Robert Kasanicky - 30/Jul/08 04:39 AM
cleaned up the JobRepository interface and DAO interfaces (there is now standalone ExecutionContextDao)