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

[email protected] commented on MAHOUT-988:
------------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4573/
-----------------------------------------------------------

Review request for mahout.


Summary
-------

Used ClusterIterator and ClusteringPolicy to buildClusters for KMeans. Removed 
KMeansClusterer, KMeansReducer, KMeansMapper and KMeansCombiner, along with 
their unit tests.


This addresses bug MAHOUT-988.
    https://issues.apache.org/jira/browse/MAHOUT-988


Diffs
-----

  
trunk/core/src/main/java/org/apache/mahout/clustering/kmeans/KMeansClusterer.java
 1307457 
  
trunk/core/src/main/java/org/apache/mahout/clustering/kmeans/KMeansCombiner.java
 1307457 
  
trunk/core/src/main/java/org/apache/mahout/clustering/kmeans/KMeansDriver.java 
1307457 
  
trunk/core/src/main/java/org/apache/mahout/clustering/kmeans/KMeansMapper.java 
1307457 
  
trunk/core/src/main/java/org/apache/mahout/clustering/kmeans/KMeansReducer.java 
1307457 
  trunk/core/src/main/java/org/apache/mahout/clustering/kmeans/KMeansUtil.java 
1307457 
  
trunk/core/src/test/java/org/apache/mahout/clustering/kmeans/TestKmeansClustering.java
 1307457 

Diff: https://reviews.apache.org/r/4573/diff


Testing
-------

All junit tests pass.


Thanks,

Paritosh


                
> Convert K-means buildClusters to use new ClusterIterator
> --------------------------------------------------------
>
>                 Key: MAHOUT-988
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-988
>             Project: Mahout
>          Issue Type: Sub-task
>          Components: Clustering
>    Affects Versions: 0.6
>            Reporter: Jeff Eastman
>            Assignee: Paritosh Ranjan
>             Fix For: 0.7
>
>
> Refactor the current K-means implementation to use the 
> ClusterIterator/Classifier implementation. This will replace the mapper, 
> combiner, reducer, clusterer and many unit tests but will not modify the 
> other driver APIs, thus retaining compatibility with existing CLI.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to