Github user debasish83 commented on the pull request:

    https://github.com/apache/spark/pull/6213#issuecomment-102843964
  
    For CosineKernel and ProductKernel, we should be able to have a separate 
code path with BLAS-2 once SparseMatrix x SparseVector merges and BLAS-3 once 
SparseMatrix x SparseMatrix merges..Basically refactor blockify from 
MatrixFactorizationModel to IndexedRowMatrix...Right now the sparse features 
are not in master yet...For Euclidean, RBF and Pearson, even with these changes 
merged, I think we still have to stay in BLAS-1


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Reply via email to