You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mahout.apache.org by sr...@apache.org on 2010/04/07 18:19:27 UTC

svn commit: r931608 - /lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/hadoop/item/RecommenderMapper.java

Author: srowen
Date: Wed Apr  7 16:19:27 2010
New Revision: 931608

URL: http://svn.apache.org/viewvc?rev=931608&view=rev
Log:
Don't output recommendations when there are none

Modified:
    lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/hadoop/item/RecommenderMapper.java

Modified: lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/hadoop/item/RecommenderMapper.java
URL: http://svn.apache.org/viewvc/lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/hadoop/item/RecommenderMapper.java?rev=931608&r1=931607&r2=931608&view=diff
==============================================================================
--- lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/hadoop/item/RecommenderMapper.java (original)
+++ lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/hadoop/item/RecommenderMapper.java Wed Apr  7 16:19:27 2010
@@ -151,11 +151,13 @@ public final class RecommenderMapper ext
         }
       }
     }
-    
-    List<RecommendedItem> recommendations = new ArrayList<RecommendedItem>(topItems.size());
-    recommendations.addAll(topItems);
-    Collections.sort(recommendations, ByValueRecommendedItemComparator.getInstance());
-    output.collect(userID, new RecommendedItemsWritable(recommendations));
+
+    if (!topItems.isEmpty()) {
+      List<RecommendedItem> recommendations = new ArrayList<RecommendedItem>(topItems.size());
+      recommendations.addAll(topItems);
+      Collections.sort(recommendations, ByValueRecommendedItemComparator.getInstance());
+      output.collect(userID, new RecommendedItemsWritable(recommendations));
+    }
   }
   
   @Override