[ https://issues.apache.org/jira/browse/MAHOUT-320?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12841187#action_12841187 ]
Sean Owen commented on MAHOUT-320: ---------------------------------- While I'm here, what's the point in the "^ 0x80" express in get/putInt()? It flips the top bit when writing, and unflips when reading, which seems like a confusing no-op. What am I missing? > Modify IntPairWritable in LDA implementation to be binary comparable to > improve performance. > -------------------------------------------------------------------------------------------- > > Key: MAHOUT-320 > URL: https://issues.apache.org/jira/browse/MAHOUT-320 > Project: Mahout > Issue Type: Improvement > Components: Clustering > Affects Versions: 0.3 > Reporter: Drew Farris > Assignee: Robin Anil > Priority: Minor > Attachments: MAHOUT-320.patch, MAHOUT-320.patch, MAHOUT-320.patch, > MAHOUT-320.patch, MAHOUT-320.patch > > > Per discussion with Robin, modifying o.a.m.clustering.lda.IntPairWritable to > be binary comparable will improve the performance of the comparison > operations during a sort because no marshaling will need to occur to compare > IntPairWritable instances. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.