Github user uzadude commented on the issue:

    https://github.com/apache/spark/pull/16732
  
    After running some more experiments I was able to reduce the runtime by 
another 1.5x factor. So currently the 
"toCoordinateMatrix().toIndexedRowMatrix()" is better by a bit only in the 
extreme cases when the block matrix size was somewhat incorrectly configured 
(as above - 1000x1000 and density 1/1000) - meaning it will contain many rows 
with only one value, then the gain comes only from the overhead of shuffling 
primitive instead of a Vector. So I generally think this approach is better.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to