[SPARK-4254] [mllib] MovieLensALS bug fix

[jkbradley]

Changed code so it does not try to serialize Params.
CC: @mengxr @debasish83 @srowen

[SparkQA]

Test build #22945 has started for PR 3116 at commit 9401b16.

• This patch merges cleanly.

[debasish83]

should we make AbstractParam serializable instead ?

[jkbradley]

I tried that first and ran into issues still. I'm not sure why it doesn't work to have Params and AbstractParams be Serializable.

[SparkQA]

Test build #22945 has finished for PR 3116 at commit 9401b16.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[AmplabJenkins]

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/22945/
Test FAILed.

[srowen]

LGTM.

[jkbradley]

Thanks! Pushed merge with master so it re-runs tests. (Failure was with YARN)

[SparkQA]

Test build #22951 has started for PR 3116 at commit e575bd8.

• This patch merges cleanly.

[SparkQA]

Test build #22951 has finished for PR 3116 at commit e575bd8.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[AmplabJenkins]

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/22951/
Test PASSed.

[mengxr]

Merged into master and branch-1.2 Thanks!