use require

nemccarthy · Jun 2, 2015 · 20a2811 · 20a2811
1 parent 6e4f8ca
commit 20a2811
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/mllib/src/main/scala/org/apache/spark/mllib/util/MLUtils.scala b/mllib/src/main/scala/org/apache/spark/mllib/util/MLUtils.scala
@@ -83,14 +83,12 @@ object MLUtils {
           (index, value)
         }.unzip
 
-        // check if indices is one-based and in ascending order
+        // check if indices are one-based and in ascending order
         var previous = -1
         var i = 0
         val indicesLength = indices.size
         while (i < indicesLength) {
-          if (indices(i) <= previous) {
-            throw new IllegalArgumentException("indices should be one-based and in ascending order")
-          }
+          require(indices(i) > previous, "indices should be one-based and in ascending order" )
           previous = indices(i)
           i += 1
         }