[MAHOUT-1560] Last batch is not filled correctly in MultithreadedBatchItemSimilarities

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 0.9
• Fix Version/s: 0.10.0
• Assignee: Sebastian Schelter
• Reporter: Jarosław Bojar
• Created: 23/May/14 23:47
• Updated: 31/Jan/24 22:12
• Resolved: 24/May/14 15:18

Description

In MultithreadedBatchItemSimilarities method queueItemIDsInBatches handles last batch incorrectly. Last batch length is calculated incorrectly. As a result last batch is either truncated or too long with superfluous positions filled with item indexes from previous batch (or zeros if it is also the first batch as in attached test).

Attached test fails for very short model (with only 4 items) with NoSuchItemException.

Attached patch corrects this issue.