[GitHub] [spark] srowen commented on pull request #41384: [SPARK-43297][ML]Use scala parallel collection ParVector to accelarate LocalKMeans.

["srowen (via GitHub)" <gi...@apache.org>]

srowen commented on PR #41384:
URL: https://github.com/apache/spark/pull/41384#issuecomment-1572017926

   I think the idea is that you're sharing a global thread pool to do this, which may have side effects and may use more cores than the driver is configured to use. It isn't terrible. But you could also just make and stop an executioncontext for this to separate it and configure it


-- 
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.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

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


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