[ 
https://issues.apache.org/jira/browse/MAHOUT-456?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12895840#action_12895840
 ] 

Sebastian Schelter commented on MAHOUT-456:
-------------------------------------------

{quote}
Would it not be better to fix that inner loop of AbstractVector.assign(), to
zero out
the current values, followed by using iterateNonZero() on the argument?

  -jake

{quote}

Added an optimized assign(Vector) method to SequentialAccessSparseVector 
instead of changing the output of the RowSimilarityJob.

> RowSimilarityJob should not produce SequentialAccessSparseVectors
> -----------------------------------------------------------------
>
>                 Key: MAHOUT-456
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-456
>             Project: Mahout
>          Issue Type: Improvement
>          Components: Math
>            Reporter: Sebastian Schelter
>         Attachments: MAHOUT-456-2.patch, MAHOUT-456.patch
>
>
> RowSimilarityJob currently produces SequentialAccessSparseVectors with 
> cardinality Integer.MAX_VALUE wrapped inside VectorWritables.
> It should better produce RandomAccessSparseVectors as some methods like 
> assign(Vector) are very slow on such SequentialAccessSparseVectors.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to