Github user wilson-lauw commented on the issue:
https://github.com/apache/spark/pull/14219
Closing this.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so
Github user wilson-lauw closed the pull request at:
https://github.com/apache/spark/pull/14219
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user wilson-lauw commented on the issue:
https://github.com/apache/spark/pull/14219
Modifications also needed on module mllib-local, file
org.apache.spark.ml.linalg.Vectors.scala. Once the changes is okay, I will also
apply the changes there.
---
If your project is set up
Github user wilson-lauw commented on the issue:
https://github.com/apache/spark/pull/14219
@hhbyyh I just checked, the implementation of `def dot(x: SparseVector, y:
SparseVector)` also need to be modified.
---
If your project is set up for it, you can reply to this email and have
Github user wilson-lauw commented on the issue:
https://github.com/apache/spark/pull/14219
file org.apache.spark.ml.linalg.BLAS.scala on module mllib-local also need
to be fixed, once the changes is okay, I will also apply the changes there.
---
If your project is set up for it, you
Github user wilson-lauw commented on a diff in the pull request:
https://github.com/apache/spark/pull/14219#discussion_r71086805
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/linalg/BLAS.scala ---
@@ -617,8 +617,27 @@ private[spark] object BLAS extends Serializable
GitHub user wilson-lauw opened a pull request:
https://github.com/apache/spark/pull/14219
sort sparseVector's indices before doing multiplication
## What changes were proposed in this pull request?
sort sparseVector's indices before doing multiplication to make sure