Bug 4167

Summary: @CheckpointInfo clarification
Product: jbatch Reporter: mminella
Component: sourceAssignee: cvignola
Status: CLOSED FIXED    
Severity: normal CC: issues
Priority: P5    
Version: 1   
Target Milestone: ---   
Hardware: All   
OS: All   

Description mminella 2012-09-28 15:30:02 UTC
@CheckpointInfo ( requires the return of an Externalizable object.  I would expect this method to just update the appropriate context (Job, Step, Chunk, etc) and let the container address anything else.
Comment 1 cvignola 2012-10-05 18:28:35 UTC
@CheckpointInfo should be used to return the data that the container should checkpoint for a given reader or writer.  This provides better encapsulation of the reader or writer function than storing it in the context.  

However, there is a persistent part of the step context, which gets stored each time a checkpoint it taken. 

If we make @CheckpointInfo optional, then the user has a choice.
Comment 2 cvignola 2013-01-16 16:03:42 UTC
this was made optional on the abstract reader/writer classes