spark git commit: [MINOR] Resolve a number of miscellaneous build warnings
Repository: spark Updated Branches: refs/heads/branch-2.0 f3570bcea -> be2d23dfd [MINOR] Resolve a number of miscellaneous build warnings This change resolves a number of build warnings that have accumulated, before 2.x. It does not address a large number of deprecation warnings, especially related to the Accumulator API. That will happen separately. Jenkins Author: Sean OwenCloses #13377 from srowen/BuildWarnings. (cherry picked from commit ce1572d16f03d383071bcc1f30ede551e8ded49f) Signed-off-by: Sean Owen Project: http://git-wip-us.apache.org/repos/asf/spark/repo Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/be2d23df Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/be2d23df Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/be2d23df Branch: refs/heads/branch-2.0 Commit: be2d23dfd97ae8b04c4960784e1223a54fd736bc Parents: f3570bc Author: Sean Owen Authored: Sun May 29 16:48:14 2016 -0500 Committer: Sean Owen Committed: Sun May 29 16:49:29 2016 -0500 -- .../unsafe/src/main/java/org/apache/spark/unsafe/Platform.java | 4 ++-- .../java/org/apache/spark/mllib/fpm/JavaPrefixSpanSuite.java | 4 +++- .../test/scala/org/apache/spark/mllib/fpm/PrefixSpanSuite.scala | 2 ++ .../org/apache/spark/sql/catalyst/ScalaReflectionSuite.scala | 2 +- .../spark/sql/catalyst/encoders/ExpressionEncoderSuite.scala | 3 ++- sql/core/src/test/scala/org/apache/spark/sql/JoinSuite.scala | 2 ++ .../spark/sql/execution/datasources/FileCatalogSuite.scala | 2 ++ .../org/apache/spark/network/yarn/YarnShuffleServiceSuite.scala | 1 + 8 files changed, 15 insertions(+), 5 deletions(-) -- http://git-wip-us.apache.org/repos/asf/spark/blob/be2d23df/common/unsafe/src/main/java/org/apache/spark/unsafe/Platform.java -- diff --git a/common/unsafe/src/main/java/org/apache/spark/unsafe/Platform.java b/common/unsafe/src/main/java/org/apache/spark/unsafe/Platform.java index bdf52f3..77c8c39 100644 --- a/common/unsafe/src/main/java/org/apache/spark/unsafe/Platform.java +++ b/common/unsafe/src/main/java/org/apache/spark/unsafe/Platform.java @@ -155,8 +155,8 @@ public final class Platform { @SuppressWarnings("unchecked") public static ByteBuffer allocateDirectBuffer(int size) { try { - Class cls = Class.forName("java.nio.DirectByteBuffer"); - Constructor constructor = cls.getDeclaredConstructor(Long.TYPE, Integer.TYPE); + Class cls = Class.forName("java.nio.DirectByteBuffer"); + Constructor constructor = cls.getDeclaredConstructor(Long.TYPE, Integer.TYPE); constructor.setAccessible(true); Field cleanerField = cls.getDeclaredField("cleaner"); cleanerField.setAccessible(true); http://git-wip-us.apache.org/repos/asf/spark/blob/be2d23df/mllib/src/test/java/org/apache/spark/mllib/fpm/JavaPrefixSpanSuite.java -- diff --git a/mllib/src/test/java/org/apache/spark/mllib/fpm/JavaPrefixSpanSuite.java b/mllib/src/test/java/org/apache/spark/mllib/fpm/JavaPrefixSpanSuite.java index 75b0ec6..32d3141 100644 --- a/mllib/src/test/java/org/apache/spark/mllib/fpm/JavaPrefixSpanSuite.java +++ b/mllib/src/test/java/org/apache/spark/mllib/fpm/JavaPrefixSpanSuite.java @@ -72,7 +72,9 @@ public class JavaPrefixSpanSuite extends SharedSparkSession { try { model.save(spark.sparkContext(), outputPath); - PrefixSpanModel newModel = PrefixSpanModel.load(spark.sparkContext(), outputPath); + @SuppressWarnings("unchecked") + PrefixSpanModel newModel = + (PrefixSpanModel) PrefixSpanModel.load(spark.sparkContext(), outputPath); JavaRDD freqSeqs = newModel.freqSequences().toJavaRDD(); List localFreqSeqs = freqSeqs.collect(); Assert.assertEquals(5, localFreqSeqs.size()); http://git-wip-us.apache.org/repos/asf/spark/blob/be2d23df/mllib/src/test/scala/org/apache/spark/mllib/fpm/PrefixSpanSuite.scala -- diff --git a/mllib/src/test/scala/org/apache/spark/mllib/fpm/PrefixSpanSuite.scala b/mllib/src/test/scala/org/apache/spark/mllib/fpm/PrefixSpanSuite.scala index 6d8c7b4..4c23763 100644 --- a/mllib/src/test/scala/org/apache/spark/mllib/fpm/PrefixSpanSuite.scala +++ b/mllib/src/test/scala/org/apache/spark/mllib/fpm/PrefixSpanSuite.scala @@ -16,6 +16,8 @@ */ package org.apache.spark.mllib.fpm +import scala.language.existentials + import org.apache.spark.SparkFunSuite import org.apache.spark.mllib.util.MLlibTestSparkContext import org.apache.spark.util.Utils
spark git commit: [MINOR] Resolve a number of miscellaneous build warnings
Repository: spark Updated Branches: refs/heads/master 472f16181 -> ce1572d16 [MINOR] Resolve a number of miscellaneous build warnings ## What changes were proposed in this pull request? This change resolves a number of build warnings that have accumulated, before 2.x. It does not address a large number of deprecation warnings, especially related to the Accumulator API. That will happen separately. ## How was this patch tested? Jenkins Author: Sean OwenCloses #13377 from srowen/BuildWarnings. Project: http://git-wip-us.apache.org/repos/asf/spark/repo Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/ce1572d1 Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/ce1572d1 Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/ce1572d1 Branch: refs/heads/master Commit: ce1572d16f03d383071bcc1f30ede551e8ded49f Parents: 472f161 Author: Sean Owen Authored: Sun May 29 16:48:14 2016 -0500 Committer: Sean Owen Committed: Sun May 29 16:48:14 2016 -0500 -- .../unsafe/src/main/java/org/apache/spark/unsafe/Platform.java | 4 ++-- .../java/org/apache/spark/mllib/fpm/JavaPrefixSpanSuite.java | 4 +++- .../test/scala/org/apache/spark/mllib/fpm/PrefixSpanSuite.scala | 2 ++ .../org/apache/spark/sql/catalyst/ScalaReflectionSuite.scala | 2 +- .../spark/sql/catalyst/encoders/ExpressionEncoderSuite.scala | 3 ++- sql/core/src/test/scala/org/apache/spark/sql/JoinSuite.scala | 2 ++ .../spark/sql/execution/datasources/FileCatalogSuite.scala | 2 ++ .../org/apache/spark/network/yarn/YarnShuffleServiceSuite.scala | 2 +- 8 files changed, 15 insertions(+), 6 deletions(-) -- http://git-wip-us.apache.org/repos/asf/spark/blob/ce1572d1/common/unsafe/src/main/java/org/apache/spark/unsafe/Platform.java -- diff --git a/common/unsafe/src/main/java/org/apache/spark/unsafe/Platform.java b/common/unsafe/src/main/java/org/apache/spark/unsafe/Platform.java index bdf52f3..77c8c39 100644 --- a/common/unsafe/src/main/java/org/apache/spark/unsafe/Platform.java +++ b/common/unsafe/src/main/java/org/apache/spark/unsafe/Platform.java @@ -155,8 +155,8 @@ public final class Platform { @SuppressWarnings("unchecked") public static ByteBuffer allocateDirectBuffer(int size) { try { - Class cls = Class.forName("java.nio.DirectByteBuffer"); - Constructor constructor = cls.getDeclaredConstructor(Long.TYPE, Integer.TYPE); + Class cls = Class.forName("java.nio.DirectByteBuffer"); + Constructor constructor = cls.getDeclaredConstructor(Long.TYPE, Integer.TYPE); constructor.setAccessible(true); Field cleanerField = cls.getDeclaredField("cleaner"); cleanerField.setAccessible(true); http://git-wip-us.apache.org/repos/asf/spark/blob/ce1572d1/mllib/src/test/java/org/apache/spark/mllib/fpm/JavaPrefixSpanSuite.java -- diff --git a/mllib/src/test/java/org/apache/spark/mllib/fpm/JavaPrefixSpanSuite.java b/mllib/src/test/java/org/apache/spark/mllib/fpm/JavaPrefixSpanSuite.java index 75b0ec6..32d3141 100644 --- a/mllib/src/test/java/org/apache/spark/mllib/fpm/JavaPrefixSpanSuite.java +++ b/mllib/src/test/java/org/apache/spark/mllib/fpm/JavaPrefixSpanSuite.java @@ -72,7 +72,9 @@ public class JavaPrefixSpanSuite extends SharedSparkSession { try { model.save(spark.sparkContext(), outputPath); - PrefixSpanModel newModel = PrefixSpanModel.load(spark.sparkContext(), outputPath); + @SuppressWarnings("unchecked") + PrefixSpanModel newModel = + (PrefixSpanModel) PrefixSpanModel.load(spark.sparkContext(), outputPath); JavaRDD freqSeqs = newModel.freqSequences().toJavaRDD(); List localFreqSeqs = freqSeqs.collect(); Assert.assertEquals(5, localFreqSeqs.size()); http://git-wip-us.apache.org/repos/asf/spark/blob/ce1572d1/mllib/src/test/scala/org/apache/spark/mllib/fpm/PrefixSpanSuite.scala -- diff --git a/mllib/src/test/scala/org/apache/spark/mllib/fpm/PrefixSpanSuite.scala b/mllib/src/test/scala/org/apache/spark/mllib/fpm/PrefixSpanSuite.scala index 6d8c7b4..4c23763 100644 --- a/mllib/src/test/scala/org/apache/spark/mllib/fpm/PrefixSpanSuite.scala +++ b/mllib/src/test/scala/org/apache/spark/mllib/fpm/PrefixSpanSuite.scala @@ -16,6 +16,8 @@ */ package org.apache.spark.mllib.fpm +import scala.language.existentials + import org.apache.spark.SparkFunSuite import org.apache.spark.mllib.util.MLlibTestSparkContext import org.apache.spark.util.Utils