Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/20459#discussion_r165239367
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/QuantileDiscretizer.scala ---
@@ -93,7 +93,7 @@ private[feature] trait QuantileDiscretizerBase extends
Params
* @group param
*/
@Since("2.1.0")
- override val handleInvalid: Param[String] = new Param[String](this,
"handleInvalid",
+ final override val handleInvalid: Param[String] = new
Param[String](this, "handleInvalid",
--- End diff --
Fair enough. I will leave ```handleInvalid``` in all estimators
```non-final```.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
