spark git commit: [MINOR] Resolve a number of miscellaneous build warnings

2016-05-29 Thread srowen
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 Owen 

Closes #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

2016-05-29 Thread srowen
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 Owen 

Closes #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