diff options
| author | Vasilis Vryniotis <bbriniotis@datumbox.com> | 2017-03-02 12:37:42 +0200 |
|---|---|---|
| committer | Nick Pentreath <nickp@za.ibm.com> | 2017-03-02 12:37:42 +0200 |
| commit | 625cfe09e673bfcb95e361ce19b534cf0a3c782c (patch) | |
| tree | 7602307619a40627ffff5f838befaf4e473a9671 /core/src | |
| parent | 8d6ef895ee492b8febbaac7ab2ef2c907b48fa4a (diff) | |
| download | spark-625cfe09e673bfcb95e361ce19b534cf0a3c782c.tar.gz spark-625cfe09e673bfcb95e361ce19b534cf0a3c782c.tar.bz2 spark-625cfe09e673bfcb95e361ce19b534cf0a3c782c.zip | |
[SPARK-19733][ML] Removed unnecessary castings and refactored checked casts in ALS.
## What changes were proposed in this pull request?
The original ALS was performing unnecessary casting to the user and item ids because the protected checkedCast() method required a double. I removed the castings and refactored the method to receive Any and efficiently handle all permitted numeric values.
## How was this patch tested?
I tested it by running the unit-tests and by manually validating the result of checkedCast for various legal and illegal values.
Author: Vasilis Vryniotis <bbriniotis@datumbox.com>
Closes #17059 from datumbox/als_casting_fix.
Diffstat (limited to 'core/src')
0 files changed, 0 insertions, 0 deletions