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 2012/02/09 15:08:59 UTC

svn commit: r1242333 - /mahout/trunk/integration/src/main/java/org/apache/mahout/utils/vectors/arff/MapBackedARFFModel.java

Author: srowen
Date: Thu Feb  9 14:08:58 2012
New Revision: 1242333

URL: http://svn.apache.org/viewvc?rev=1242333&view=rev
Log:
MAHOUT-948 better error for bad type

Modified:
    mahout/trunk/integration/src/main/java/org/apache/mahout/utils/vectors/arff/MapBackedARFFModel.java

Modified: mahout/trunk/integration/src/main/java/org/apache/mahout/utils/vectors/arff/MapBackedARFFModel.java
URL: http://svn.apache.org/viewvc/mahout/trunk/integration/src/main/java/org/apache/mahout/utils/vectors/arff/MapBackedARFFModel.java?rev=1242333&r1=1242332&r2=1242333&view=diff
==============================================================================
--- mahout/trunk/integration/src/main/java/org/apache/mahout/utils/vectors/arff/MapBackedARFFModel.java (original)
+++ mahout/trunk/integration/src/main/java/org/apache/mahout/utils/vectors/arff/MapBackedARFFModel.java Thu Feb  9 14:08:58 2012
@@ -84,6 +84,9 @@ public class MapBackedARFFModel implemen
   @Override
   public double getValue(String data, int idx) {
     ARFFType type = typeMap.get(idx);
+    if (type == null) {
+      throw new IllegalArgumentException("Attribute type cannot be NULL, attribute index was: " + idx);
+    }
     data = QUOTE_PATTERN.matcher(data).replaceAll("");
     data = data.trim();
     double result;