Skip to content

[SPARK-16440] [MLLIB] Undeleted broadcast variables in Word2Vec causing OoM for long runs #14153

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

srowen
Copy link
Member

@srowen srowen commented Jul 12, 2016

What changes were proposed in this pull request?

Unpersist broadcasted vars in Word2Vec.fit for more timely / reliable resource cleanup

How was this patch tested?

Jenkins tests

@SparkQA
Copy link

SparkQA commented Jul 12, 2016

Test build #62165 has finished for PR 14153 at commit 0a933d8.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

asfgit pushed a commit that referenced this pull request Jul 13, 2016
…g OoM for long runs

## What changes were proposed in this pull request?

Unpersist broadcasted vars in Word2Vec.fit for more timely / reliable resource cleanup

## How was this patch tested?

Jenkins tests

Author: Sean Owen <sowen@cloudera.com>

Closes #14153 from srowen/SPARK-16440.

(cherry picked from commit 51ade51)
Signed-off-by: Sean Owen <sowen@cloudera.com>
@srowen
Copy link
Member Author

srowen commented Jul 13, 2016

Merged to master/2.0/1.6

asfgit pushed a commit that referenced this pull request Jul 13, 2016
…g OoM for long runs

## What changes were proposed in this pull request?

Unpersist broadcasted vars in Word2Vec.fit for more timely / reliable resource cleanup

## How was this patch tested?

Jenkins tests

Author: Sean Owen <sowen@cloudera.com>

Closes #14153 from srowen/SPARK-16440.

(cherry picked from commit 51ade51)
Signed-off-by: Sean Owen <sowen@cloudera.com>
@asfgit asfgit closed this in 51ade51 Jul 13, 2016
@srowen srowen deleted the SPARK-16440 branch July 13, 2016 10:42
zzcclp pushed a commit to zzcclp/spark that referenced this pull request Jul 13, 2016
…g OoM for long runs

## What changes were proposed in this pull request?

Unpersist broadcasted vars in Word2Vec.fit for more timely / reliable resource cleanup

## How was this patch tested?

Jenkins tests

Author: Sean Owen <sowen@cloudera.com>

Closes apache#14153 from srowen/SPARK-16440.

(cherry picked from commit 51ade51)
Signed-off-by: Sean Owen <sowen@cloudera.com>
(cherry picked from commit 4381e21)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants