Repository: spark Updated Branches: refs/heads/master 26d7af911 -> e9e1adc03
[MINOR][ML] Fixed MLlib build warnings ## What changes were proposed in this pull request? Fixes to eliminate warnings during package and doc builds. ## How was this patch tested? Existing unit tests Author: Joseph K. Bradley <jos...@databricks.com> Closes #12263 from jkbradley/warning-cleanups. Project: http://git-wip-us.apache.org/repos/asf/spark/repo Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/e9e1adc0 Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/e9e1adc0 Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/e9e1adc0 Branch: refs/heads/master Commit: e9e1adc036643c6b126237903b8e79ab379b1d32 Parents: 26d7af9 Author: Joseph K. Bradley <jos...@databricks.com> Authored: Tue Apr 12 03:24:26 2016 +0100 Committer: Sean Owen <so...@cloudera.com> Committed: Tue Apr 12 03:24:26 2016 +0100 ---------------------------------------------------------------------- .../spark/examples/mllib/JavaStratifiedSamplingExample.java | 1 + .../scala/org/apache/spark/ml/tree/impl/RandomForestSuite.scala | 3 +++ 2 files changed, 4 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/spark/blob/e9e1adc0/examples/src/main/java/org/apache/spark/examples/mllib/JavaStratifiedSamplingExample.java ---------------------------------------------------------------------- diff --git a/examples/src/main/java/org/apache/spark/examples/mllib/JavaStratifiedSamplingExample.java b/examples/src/main/java/org/apache/spark/examples/mllib/JavaStratifiedSamplingExample.java index 86c389e..72bbb2a 100644 --- a/examples/src/main/java/org/apache/spark/examples/mllib/JavaStratifiedSamplingExample.java +++ b/examples/src/main/java/org/apache/spark/examples/mllib/JavaStratifiedSamplingExample.java @@ -35,6 +35,7 @@ public class JavaStratifiedSamplingExample { SparkConf conf = new SparkConf().setAppName("JavaStratifiedSamplingExample"); JavaSparkContext jsc = new JavaSparkContext(conf); + @SuppressWarnings("unchecked") // $example on$ List<Tuple2<Integer, Character>> list = new ArrayList<>( Arrays.<Tuple2<Integer, Character>>asList( http://git-wip-us.apache.org/repos/asf/spark/blob/e9e1adc0/mllib/src/test/scala/org/apache/spark/ml/tree/impl/RandomForestSuite.scala ---------------------------------------------------------------------- diff --git a/mllib/src/test/scala/org/apache/spark/ml/tree/impl/RandomForestSuite.scala b/mllib/src/test/scala/org/apache/spark/ml/tree/impl/RandomForestSuite.scala index e64551f..cd402b1 100644 --- a/mllib/src/test/scala/org/apache/spark/ml/tree/impl/RandomForestSuite.scala +++ b/mllib/src/test/scala/org/apache/spark/ml/tree/impl/RandomForestSuite.scala @@ -327,7 +327,9 @@ class RandomForestSuite extends SparkFunSuite with MLlibTestSparkContext { case n: InternalNode => n.split match { case s: CategoricalSplit => assert(s.leftCategories === Array(1.0)) + case _ => throw new AssertionError("model.rootNode.split was not a CategoricalSplit") } + case _ => throw new AssertionError("model.rootNode was not an InternalNode") } } @@ -352,6 +354,7 @@ class RandomForestSuite extends SparkFunSuite with MLlibTestSparkContext { assert(n.leftChild.isInstanceOf[InternalNode]) assert(n.rightChild.isInstanceOf[InternalNode]) Array(n.leftChild.asInstanceOf[InternalNode], n.rightChild.asInstanceOf[InternalNode]) + case _ => throw new AssertionError("rootNode was not an InternalNode") } // Single group second level tree construction. --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org For additional commands, e-mail: commits-h...@spark.apache.org