You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by ml...@apache.org on 2016/06/02 23:33:19 UTC
spark git commit: [MINOR] clean up style for storage param setters in
ALS
Repository: spark
Updated Branches:
refs/heads/master 985d53281 -> ccd298eb6
[MINOR] clean up style for storage param setters in ALS
Clean up style for param setter methods in ALS to match standard style and the other setter in class (this is an artefact of one of my previous PRs that wasn't cleaned up).
## How was this patch tested?
Existing tests - no functionality change.
Author: Nick Pentreath <ni...@za.ibm.com>
Closes #13480 from MLnick/als-param-minor-cleanup.
Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/ccd298eb
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/ccd298eb
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/ccd298eb
Branch: refs/heads/master
Commit: ccd298eb6794cbcb220ac9889db60d745231e0fe
Parents: 985d532
Author: Nick Pentreath <ni...@za.ibm.com>
Authored: Thu Jun 2 16:33:16 2016 -0700
Committer: Nick Pentreath <ni...@za.ibm.com>
Committed: Thu Jun 2 16:33:16 2016 -0700
----------------------------------------------------------------------
.../main/scala/org/apache/spark/ml/recommendation/ALS.scala | 8 ++------
1 file changed, 2 insertions(+), 6 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/spark/blob/ccd298eb/mllib/src/main/scala/org/apache/spark/ml/recommendation/ALS.scala
----------------------------------------------------------------------
diff --git a/mllib/src/main/scala/org/apache/spark/ml/recommendation/ALS.scala b/mllib/src/main/scala/org/apache/spark/ml/recommendation/ALS.scala
index 8dc7437..2404a69 100644
--- a/mllib/src/main/scala/org/apache/spark/ml/recommendation/ALS.scala
+++ b/mllib/src/main/scala/org/apache/spark/ml/recommendation/ALS.scala
@@ -430,15 +430,11 @@ class ALS(@Since("1.4.0") override val uid: String) extends Estimator[ALSModel]
/** @group expertSetParam */
@Since("2.0.0")
- def setIntermediateStorageLevel(value: String): this.type = {
- set(intermediateStorageLevel, value)
- }
+ def setIntermediateStorageLevel(value: String): this.type = set(intermediateStorageLevel, value)
/** @group expertSetParam */
@Since("2.0.0")
- def setFinalStorageLevel(value: String): this.type = {
- set(finalStorageLevel, value)
- }
+ def setFinalStorageLevel(value: String): this.type = set(finalStorageLevel, value)
/**
* Sets both numUserBlocks and numItemBlocks to the specific value.
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org