[
https://issues.apache.org/jira/browse/MAHOUT-1091?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13472593#comment-13472593
]
Ted Dunning commented on MAHOUT-1091:
-------------------------------------
The test that I saw in your patch creates a sparse vector that is MAX_INTEGER
in size, but you only verify that the iterator goes through the first few
elements. It is better to create a smaller vector and verify that it scans
through all of the elements, both zero and non-zero.
Take a look at the test in my patch.
> Bug in SequentialAccessSparseVector full iteration
> --------------------------------------------------
>
> Key: MAHOUT-1091
> URL: https://issues.apache.org/jira/browse/MAHOUT-1091
> Project: Mahout
> Issue Type: Bug
> Components: Math
> Reporter: Ted Dunning
> Fix For: 0.8
>
> Attachments:
> 0001-MAHOUT-1091-Add-test-to-demonstrate-broken-iterator-.patch,
> 0001-MAHOUT-1091-Add-test-to-demonstrate-broken-iterator-.patch,
> MAHOUT-1091.patch
>
>
> The iterator for the SequentialAccessSparseVector doesn't return any items
> beyond the last non-zero. This breaks some stuff pretty massively, but
> hopefully doesn't break much user code since iterating through all elements
> of a sparse vector is a relatively rare thing to do.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
