[BATCHEE-48] ItemProcessListener.onProcessError() gets called with 'processedItem' instead of 'itemRead'

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.2-incubating
• Fix Version/s: 0.2-incubating
• Assignee: Romain Manni-Bucau
• Reporter: Alonso Gonzalez
• Created: 24/Jul/14 07:49
• Updated: 07/Dec/15 22:40
• Resolved: 24/Jul/14 17:17

Description

In case of an exception, ChunkStepController calls ItemProcessListener.onProcessError with 'processedItem'. But processedItem will be always 'null' unless the process method finishes without an exception.

The specification states that 'the item about to be processed' should be passed:

/**

• The afterProcess method receives control after an item
• processor processes an item. The method receives the item processed
• and the result item as an input.
• @param item specifies the item about to be processed.
• @param ex specifies the exception thrown by the item processor.
• @throws Exception
  */
  public void onProcessError(Object item, Exception ex) throws Exception;

(Obviously the first part of the JavaDoc has been accidentally copied from the 'afterProcess' method.)