Skip to content

Conversation

@tdas
Copy link
Contributor

@tdas tdas commented May 4, 2016

What changes were proposed in this pull request?

Make sure that whenever the StateStoreCoordinator cannot be contacted, assume that the SparkContext and RpcEnv on the driver has been shutdown, and therefore stop the StateStore management thread, and unload all loaded stores.

How was this patch tested?

Updated unit tests.

@tdas
Copy link
Contributor Author

tdas commented May 4, 2016

@zsxwing

@SparkQA
Copy link

SparkQA commented May 4, 2016

Test build #57788 has finished for PR 12905 at commit a8cb982.

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

@zsxwing
Copy link
Member

zsxwing commented May 4, 2016

retest this please

2 similar comments
@zsxwing
Copy link
Member

zsxwing commented May 4, 2016

retest this please

@zsxwing
Copy link
Member

zsxwing commented May 4, 2016

retest this please

@zsxwing
Copy link
Member

zsxwing commented May 4, 2016

Let's run more times to see if it's stable.

@SparkQA
Copy link

SparkQA commented May 4, 2016

Test build #57804 has finished for PR 12905 at commit a8cb982.

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

@SparkQA
Copy link

SparkQA commented May 4, 2016

Test build #2970 has finished for PR 12905 at commit a8cb982.

  • This patch fails MiMa tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@tdas
Copy link
Contributor Author

tdas commented May 4, 2016

Something flaky with Mima!

@SparkQA
Copy link

SparkQA commented May 5, 2016

Test build #2969 has finished for PR 12905 at commit a8cb982.

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

@SparkQA
Copy link

SparkQA commented May 5, 2016

Test build #2971 has finished for PR 12905 at commit a8cb982.

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

@SparkQA
Copy link

SparkQA commented May 5, 2016

Test build #2972 has finished for PR 12905 at commit a8cb982.

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

@SparkQA
Copy link

SparkQA commented May 5, 2016

Test build #2973 has finished for PR 12905 at commit a8cb982.

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

@zsxwing
Copy link
Member

zsxwing commented May 5, 2016

LGTM. Merging to master / 2.0

asfgit pushed a commit that referenced this pull request May 5, 2016
…ntext has been shutdown

## What changes were proposed in this pull request?

Make sure that whenever the StateStoreCoordinator cannot be contacted, assume that the SparkContext and RpcEnv on the driver has been shutdown, and therefore stop the StateStore management thread, and unload all loaded stores.

## How was this patch tested?

Updated unit tests.

Author: Tathagata Das <[email protected]>

Closes #12905 from tdas/SPARK-15131.

(cherry picked from commit bde27b8)
Signed-off-by: Shixiong Zhu <[email protected]>
@asfgit asfgit closed this in bde27b8 May 5, 2016
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