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 2011/06/04 18:31:51 UTC

svn commit: r1131429 - /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/model/mongodb/MongoDBDataModel.java

Author: srowen
Date: Sat Jun  4 16:31:50 2011
New Revision: 1131429

URL: http://svn.apache.org/viewvc?rev=1131429&view=rev
Log:
Follow up on MAHOUT-705, additional check for empty args

Modified:
    mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/model/mongodb/MongoDBDataModel.java

Modified: mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/model/mongodb/MongoDBDataModel.java
URL: http://svn.apache.org/viewvc/mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/model/mongodb/MongoDBDataModel.java?rev=1131429&r1=1131428&r2=1131429&view=diff
==============================================================================
--- mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/model/mongodb/MongoDBDataModel.java (original)
+++ mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/model/mongodb/MongoDBDataModel.java Sat Jun  4 16:31:50 2011
@@ -748,8 +748,10 @@ public final class MongoDBDataModel impl
                          boolean add) throws NoSuchUserException, NoSuchItemException {
     Preconditions.checkNotNull(userID);
     Preconditions.checkNotNull(items);
+    Preconditions.checkArgument(userID.length() > 0);
     for (List<String> item : items) {
       Preconditions.checkNotNull(item.get(0));
+      Preconditions.checkArgument(item.get(0).length() > 0);
     }
     if (userIsObject && !ID_PATTERN.matcher(userID).matches()) {
       throw new IllegalArgumentException();