Skip to content
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

[MINOR][CORE] Show block manager id when remove RDD/Broadcast fails. #20960

Closed
wants to merge 1 commit into from

Conversation

jiangxb1987
Copy link
Contributor

What changes were proposed in this pull request?

Address #20924 (comment), show block manager id when remove RDD/Broadcast fails.

How was this patch tested?

N/A

@SparkQA
Copy link

SparkQA commented Apr 2, 2018

Test build #88819 has finished for PR 20960 at commit 1109720.

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

@HyukjinKwon
Copy link
Member

cc @viirya too before merging it.

Copy link
Member

@viirya viirya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@HyukjinKwon
Copy link
Member

Merged to master and branch-2.3.

asfgit pushed a commit that referenced this pull request Apr 3, 2018
## What changes were proposed in this pull request?

Address #20924 (comment), show block manager id when remove RDD/Broadcast fails.

## How was this patch tested?

N/A

Author: Xingbo Jiang <[email protected]>

Closes #20960 from jiangxb1987/bmid.

(cherry picked from commit 7cf9fab)
Signed-off-by: hyukjinkwon <[email protected]>
@asfgit asfgit closed this in 7cf9fab Apr 3, 2018
robert3005 pushed a commit to palantir/spark that referenced this pull request Apr 4, 2018
## What changes were proposed in this pull request?

Address apache#20924 (comment), show block manager id when remove RDD/Broadcast fails.

## How was this patch tested?

N/A

Author: Xingbo Jiang <[email protected]>

Closes apache#20960 from jiangxb1987/bmid.
mshtelma pushed a commit to mshtelma/spark that referenced this pull request Apr 5, 2018
## What changes were proposed in this pull request?

Address apache#20924 (comment), show block manager id when remove RDD/Broadcast fails.

## How was this patch tested?

N/A

Author: Xingbo Jiang <[email protected]>

Closes apache#20960 from jiangxb1987/bmid.
peter-toth pushed a commit to peter-toth/spark that referenced this pull request Oct 6, 2018
## What changes were proposed in this pull request?

Address apache#20924 (comment), show block manager id when remove RDD/Broadcast fails.

## How was this patch tested?

N/A

Author: Xingbo Jiang <[email protected]>

Closes apache#20960 from jiangxb1987/bmid.

(cherry picked from commit 7cf9fab)
Signed-off-by: hyukjinkwon <[email protected]>
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.

5 participants