Skip to content

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 <[email protected]>

Closes #14153 from srowen/SPARK-16440.

(cherry picked from commit 51ade51)
Signed-off-by: Sean Owen <[email protected]>
@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 <[email protected]>

Closes #14153 from srowen/SPARK-16440.

(cherry picked from commit 51ade51)
Signed-off-by: Sean Owen <[email protected]>
@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 <[email protected]>

Closes apache#14153 from srowen/SPARK-16440.

(cherry picked from commit 51ade51)
Signed-off-by: Sean Owen <[email protected]>
(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