From ccd298eb6794cbcb220ac9889db60d745231e0fe Mon Sep 17 00:00:00 2001
From: Nick Pentreath <nickp@za.ibm.com>
Date: Thu, 2 Jun 2016 16:33:16 -0700
Subject: [PATCH] [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 <nickp@za.ibm.com>
Closes #13480 from MLnick/als-param-minor-cleanup.
---
.../scala/org/apache/spark/ml/recommendation/ALS.scala | 8 ++------
1 file changed, 2 insertions(+), 6 deletions(-)
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 8dc7437d47..2404a69e9e 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.
--
GitLab