You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@systemml.apache.org by de...@apache.org on 2016/08/31 18:27:23 UTC

incubator-systemml git commit: [HOTFIX] Fix build for Scala Map MLContext input

Repository: incubator-systemml
Updated Branches:
  refs/heads/master b133401b4 -> b4d9dcdaa


[HOTFIX] Fix build for Scala Map MLContext input


Project: http://git-wip-us.apache.org/repos/asf/incubator-systemml/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-systemml/commit/b4d9dcda
Tree: http://git-wip-us.apache.org/repos/asf/incubator-systemml/tree/b4d9dcda
Diff: http://git-wip-us.apache.org/repos/asf/incubator-systemml/diff/b4d9dcda

Branch: refs/heads/master
Commit: b4d9dcdaa4bddfd457085e3b17249e99172f4f14
Parents: b133401
Author: Deron Eriksson <de...@us.ibm.com>
Authored: Wed Aug 31 11:25:04 2016 -0700
Committer: Deron Eriksson <de...@us.ibm.com>
Committed: Wed Aug 31 11:25:04 2016 -0700

----------------------------------------------------------------------
 .../org/apache/sysml/api/mlcontext/Script.java  | 25 ++++++++++----------
 1 file changed, 12 insertions(+), 13 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-systemml/blob/b4d9dcda/src/main/java/org/apache/sysml/api/mlcontext/Script.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/sysml/api/mlcontext/Script.java b/src/main/java/org/apache/sysml/api/mlcontext/Script.java
index 4269c86..1c2d795 100644
--- a/src/main/java/org/apache/sysml/api/mlcontext/Script.java
+++ b/src/main/java/org/apache/sysml/api/mlcontext/Script.java
@@ -246,11 +246,11 @@ public class Script {
 	/**
 	 * Pass a Scala Map of inputs to the script.
 	 * <p>
-	 * Note that the {@code Map} key/value types are not explicitly specified on
-	 * this method because {@code [String, Any]} can't be recognized on the Java
-	 * side since {@code Any} doesn't have an equivalent in the Java class
-	 * hierarchy ({@code scala.Any} is a superclass of {@code scala.AnyRef},
-	 * which is equivalent to {@code java.lang.Object}). Therefore, specifying
+	 * Note that the {@code Map} value type is not explicitly specified on this
+	 * method because {@code [String, Any]} can't be recognized on the Java side
+	 * since {@code Any} doesn't have an equivalent in the Java class hierarchy
+	 * ({@code scala.Any} is a superclass of {@code scala.AnyRef}, which is
+	 * equivalent to {@code java.lang.Object}). Therefore, specifying
 	 * {@code scala.collection.Map<String, Object>} as an input parameter to
 	 * this Java method is not encompassing enough and would require types such
 	 * as a {@code scala.Double} to be cast using {@code asInstanceOf[AnyRef]}.
@@ -260,7 +260,7 @@ public class Script {
 	 * @return {@code this} Script object to allow chaining of methods
 	 */
 	@SuppressWarnings({ "rawtypes", "unchecked" })
-	public Script in(scala.collection.Map inputs) {
+	public Script in(scala.collection.Map<String, ?> inputs) {
 		Map javaMap = JavaConversions.mapAsJavaMap(inputs);
 		in(javaMap);
 
@@ -270,11 +270,11 @@ public class Script {
 	/**
 	 * Pass a Scala Map of inputs to the script.
 	 * <p>
-	 * Note that the {@code Map} key/value types are not explicitly specified on
-	 * this method because {@code [String, Any]} can't be recognized on the Java
-	 * side since {@code Any} doesn't have an equivalent in the Java class
-	 * hierarchy ({@code scala.Any} is a superclass of {@code scala.AnyRef},
-	 * which is equivalent to {@code java.lang.Object}). Therefore, specifying
+	 * Note that the {@code Map} value type is not explicitly specified on this
+	 * method because {@code [String, Any]} can't be recognized on the Java side
+	 * since {@code Any} doesn't have an equivalent in the Java class hierarchy
+	 * ({@code scala.Any} is a superclass of {@code scala.AnyRef}, which is
+	 * equivalent to {@code java.lang.Object}). Therefore, specifying
 	 * {@code scala.collection.Map<String, Object>} as an input parameter to
 	 * this Java method is not encompassing enough and would require types such
 	 * as a {@code scala.Double} to be cast using {@code asInstanceOf[AnyRef]}.
@@ -283,8 +283,7 @@ public class Script {
 	 *            Scala Map of inputs (parameters ($) and variables).
 	 * @return {@code this} Script object to allow chaining of methods
 	 */
-	@SuppressWarnings("rawtypes")
-	public Script input(scala.collection.Map inputs) {
+	public Script input(scala.collection.Map<String, ?> inputs) {
 		return in(inputs);
 	}