Skip to content

Conversation

@jeanlyn
Copy link
Contributor

@jeanlyn jeanlyn commented Apr 4, 2016

What changes were proposed in this pull request?

This patch try to update the updatedBlockStatuses when removing blocks, making sure BlockManager correctly updates updatedBlockStatuses

How was this patch tested?

test("updated block statuses") in BlockManagerSuite.scala

@jeanlyn
Copy link
Contributor Author

jeanlyn commented Apr 4, 2016

/cc @andrewor14 .

@SparkQA
Copy link

SparkQA commented Apr 4, 2016

Test build #54853 has finished for PR 12150 at commit ff151af.

  • This patch fails to build.
  • This patch merges cleanly.
  • This patch adds no public classes.

@jeanlyn
Copy link
Contributor Author

jeanlyn commented Apr 4, 2016

retest this please.

@SparkQA
Copy link

SparkQA commented Apr 4, 2016

Test build #54854 has finished for PR 12150 at commit ff151af.

  • This patch fails from timeout after a configured wait of 250m.
  • This patch merges cleanly.
  • This patch adds no public classes.

@andrewor14
Copy link
Contributor

retest this please

@SparkQA
Copy link

SparkQA commented Apr 4, 2016

Test build #54888 has finished for PR 12150 at commit ff151af.

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

@andrewor14
Copy link
Contributor

Merging into 1.6

asfgit pushed a commit that referenced this pull request Apr 5, 2016
…locks

## What changes were proposed in this pull request?

This patch try to  update the `updatedBlockStatuses ` when removing blocks, making sure `BlockManager` correctly updates `updatedBlockStatuses`

## How was this patch tested?

test("updated block statuses") in BlockManagerSuite.scala

Author: jeanlyn <[email protected]>

Closes #12150 from jeanlyn/updataBlock1.6.
@andrewor14
Copy link
Contributor

@jeanlyn can you close this PR now that it's merged?

@jeanlyn
Copy link
Contributor Author

jeanlyn commented Apr 6, 2016

OK.

@jeanlyn jeanlyn closed this Apr 6, 2016
zzcclp pushed a commit to zzcclp/spark that referenced this pull request Apr 6, 2016
…locks

## What changes were proposed in this pull request?

This patch try to  update the `updatedBlockStatuses ` when removing blocks, making sure `BlockManager` correctly updates `updatedBlockStatuses`

## How was this patch tested?

test("updated block statuses") in BlockManagerSuite.scala

Author: jeanlyn <[email protected]>

Closes apache#12150 from jeanlyn/updataBlock1.6.

(cherry picked from commit cfe9f02)
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.

3 participants