You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by me...@apache.org on 2015/07/23 08:27:28 UTC
spark git commit: [SPARK-9268] [ML] Removed varargs annotation from
Params.setDefault taking multiple params
Repository: spark
Updated Branches:
refs/heads/master 2f5cbd860 -> 410dd41cf
[SPARK-9268] [ML] Removed varargs annotation from Params.setDefault taking multiple params
Removed varargs annotation from Params.setDefault taking multiple params.
Though varargs is technically correct, it often requires that developers do clean assembly, rather than (not clean) assembly, which is a nuisance during development.
CC: mengxr
Author: Joseph K. Bradley <jo...@databricks.com>
Closes #7604 from jkbradley/params-setdefault-varargs and squashes the following commits:
6016dc6 [Joseph K. Bradley] removed varargs annotation from Params.setDefault taking multiple params
Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/410dd41c
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/410dd41c
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/410dd41c
Branch: refs/heads/master
Commit: 410dd41cf6618b93b6daa6147d17339deeaa49ae
Parents: 2f5cbd8
Author: Joseph K. Bradley <jo...@databricks.com>
Authored: Wed Jul 22 23:27:25 2015 -0700
Committer: Xiangrui Meng <me...@databricks.com>
Committed: Wed Jul 22 23:27:25 2015 -0700
----------------------------------------------------------------------
mllib/src/main/scala/org/apache/spark/ml/param/params.scala | 5 ++++-
.../src/test/java/org/apache/spark/ml/param/JavaTestParams.java | 3 ---
2 files changed, 4 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/spark/blob/410dd41c/mllib/src/main/scala/org/apache/spark/ml/param/params.scala
----------------------------------------------------------------------
diff --git a/mllib/src/main/scala/org/apache/spark/ml/param/params.scala b/mllib/src/main/scala/org/apache/spark/ml/param/params.scala
index 824efa5..954aa17 100644
--- a/mllib/src/main/scala/org/apache/spark/ml/param/params.scala
+++ b/mllib/src/main/scala/org/apache/spark/ml/param/params.scala
@@ -476,11 +476,14 @@ trait Params extends Identifiable with Serializable {
/**
* Sets default values for a list of params.
*
+ * Note: Java developers should use the single-parameter [[setDefault()]].
+ * Annotating this with varargs can cause compilation failures due to a Scala compiler bug.
+ * See SPARK-9268.
+ *
* @param paramPairs a list of param pairs that specify params and their default values to set
* respectively. Make sure that the params are initialized before this method
* gets called.
*/
- @varargs
protected final def setDefault(paramPairs: ParamPair[_]*): this.type = {
paramPairs.foreach { p =>
setDefault(p.param.asInstanceOf[Param[Any]], p.value)
http://git-wip-us.apache.org/repos/asf/spark/blob/410dd41c/mllib/src/test/java/org/apache/spark/ml/param/JavaTestParams.java
----------------------------------------------------------------------
diff --git a/mllib/src/test/java/org/apache/spark/ml/param/JavaTestParams.java b/mllib/src/test/java/org/apache/spark/ml/param/JavaTestParams.java
index 3ae09d3..dc6ce80 100644
--- a/mllib/src/test/java/org/apache/spark/ml/param/JavaTestParams.java
+++ b/mllib/src/test/java/org/apache/spark/ml/param/JavaTestParams.java
@@ -96,11 +96,8 @@ public class JavaTestParams extends JavaParams {
new DoubleArrayParam(this, "myDoubleArrayParam", "this is a double param");
setDefault(myIntParam(), 1);
- setDefault(myIntParam().w(1));
setDefault(myDoubleParam(), 0.5);
- setDefault(myIntParam().w(1), myDoubleParam().w(0.5));
setDefault(myDoubleArrayParam(), new double[] {1.0, 2.0});
- setDefault(myDoubleArrayParam().w(new double[] {1.0, 2.0}));
}
@Override
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org