svn commit: r1196582 - /mahout/trunk/core/src/main/java/org/apache/mahout/clustering/minhash/MinHashMapper.java

[gsingers]

Author: gsingers
Date: Wed Nov  2 13:48:15 2011
New Revision: 1196582

URL: http://svn.apache.org/viewvc?rev=1196582&view=rev
Log:
minor tweak to avoid some extra string manipulation

Modified:
    
mahout/trunk/core/src/main/java/org/apache/mahout/clustering/minhash/MinHashMapper.java

Modified: 
mahout/trunk/core/src/main/java/org/apache/mahout/clustering/minhash/MinHashMapper.java
URL: 
http://svn.apache.org/viewvc/mahout/trunk/core/src/main/java/org/apache/mahout/clustering/minhash/MinHashMapper.java?rev=1196582&r1=1196581&r2=1196582&view=diff
==============================================================================
--- 
mahout/trunk/core/src/main/java/org/apache/mahout/clustering/minhash/MinHashMapper.java
 (original)
+++ 
mahout/trunk/core/src/main/java/org/apache/mahout/clustering/minhash/MinHashMapper.java
 Wed Nov  2 13:48:15 2011
@@ -102,9 +102,8 @@ public class MinHashMapper extends Mappe
       for (int j = 0; j < keyGroups; j++) {
         clusterIdBuilder.append(minHashValues[(i + j) % 
numHashFunctions]).append('-');
       }
-      String clusterId = clusterIdBuilder.toString();
-      clusterId = clusterId.substring(0, clusterId.lastIndexOf('-'));
-      Text cluster = new Text(clusterId);
+      clusterIdBuilder.deleteCharAt(clusterIdBuilder.length() - 1);
+      Text cluster = new Text(clusterIdBuilder.toString());
       Writable point;
       if (debugOutput) {
         point = new VectorWritable(featureVector.clone());