You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@spark.apache.org by "Aman Omer (Jira)" <ji...@apache.org> on 2019/11/09 18:17:00 UTC
[jira] [Commented] (SPARK-29809) Missing persist in Word2Vec.fit()
[ https://issues.apache.org/jira/browse/SPARK-29809?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16970910#comment-16970910 ]
Aman Omer commented on SPARK-29809:
-----------------------------------
Thanks [~spark_cachecheck] for reporting. I will raise a PR for this.
> Missing persist in Word2Vec.fit()
> ---------------------------------
>
> Key: SPARK-29809
> URL: https://issues.apache.org/jira/browse/SPARK-29809
> Project: Spark
> Issue Type: Improvement
> Components: MLlib
> Affects Versions: 2.4.3
> Reporter: Dong Wang
> Priority: Major
>
> The RDD dataset is used by more than two actions in learnVocab(dataset) and doFit. It needs to be persisted.
> {code:scala}
> def fit[S <: Iterable[String]](dataset: RDD[S]): Word2VecModel = {
> // Needs to persist dataset here
> learnVocab(dataset) // has action on dataset
> createBinaryTree()
> val sc = dataset.context
> val expTable = sc.broadcast(createExpTable())
> val bcVocab = sc.broadcast(vocab)
> val bcVocabHash = sc.broadcast(vocabHash)
> try {
> doFit(dataset, sc, expTable, bcVocab, bcVocabHash) // has action on dataset
> {code}
> This issue is reported by our tool _CacheCheck_, which is used to dynamically detecting persist()/unpersist() api misuses.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@spark.apache.org
For additional commands, e-mail: issues-help@spark.apache.org