zhengruifeng commented on pull request #28229:
URL: https://github.com/apache/spark/pull/28229#issuecomment-620990814


   @xwu99 My previous works include:
   LinearSVC:  https://github.com/apache/spark/pull/27360
   LogisticRegression: https://github.com/apache/spark/pull/27374
   LinearRegression: https://github.com/apache/spark/pull/27396
   GaussianMixture: https://github.com/apache/spark/pull/27473
   KMeans: 
https://github.com/apache/spark/compare/master...zhengruifeng:blockify_km?expand=1,
 not send
   
   I'm reworking on 
`LinearSVC`/`LogisticRegression`/`LinearRegression`/`GaussianMixture`. For 
KMeans I am glad you can take it over.
   
   I just recreate a new [PR](https://github.com/apache/spark/pull/28349) for 
LinearSVC, the main idea is to use expert param `blockSize` to choose the path. 
The original path will be choosen by default to avoid performance regression on 
sparse datasets.
   
   If nobody object, I will merge it, and then other three impls (since they 
depend on the first one, I do not recreate PRs right now) 
`LogisticRegression`/`LinearRegression`/`GaussianMixture`.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



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

Reply via email to