[SPARK-5609] PythonMLlibAPI trainGaussianMixture seed should use Java type

• Type: Bug
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 1.3.0
• Fix Version/s: 1.3.0
• Component/s: MLlib, PySpark
• Target Version/s: 1.3.0
• Assignee: Xiangrui Meng
• Reporter: Joseph K. Bradley
• Created: 05/Feb/15 03:10
• Updated: 05/Feb/15 05:43
• Resolved: 05/Feb/15 05:43

Description

trainGaussianMixture takes parameter seed of type scala.Long but should take java.lang.Long.
Otherwise, the test for whether seed is null (None in Python) will be ineffective. See compilation warning:

[warn] /Users/josephkb/spark/mllib/src/main/scala/org/apache/spark/mllib/api/python/PythonMLLibAPI.scala:304: comparing values of types Long and Null using `!=' will always yield true
[warn]     if (seed != null) gmmAlg.setSeed(seed)
[warn]              ^