Skip to content

Conversation

@andrewor14
Copy link
Contributor

If the cleaner is stopped, we shouldn't print a huge stack trace when the cleaner thread is interrupted because we purposefully did this.

@SparkQA
Copy link

SparkQA commented Mar 4, 2015

Test build #28248 has started for PR 4882 at commit 8652120.

  • This patch merges cleanly.

@vanzin
Copy link
Contributor

vanzin commented Mar 4, 2015

LGTM.

@SparkQA
Copy link

SparkQA commented Mar 4, 2015

Test build #28248 has finished for PR 4882 at commit 8652120.

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

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28248/
Test PASSed.

@asfgit asfgit closed this in d334bfb Mar 4, 2015
@andrewor14 andrewor14 deleted the cleaner-interrupt branch March 4, 2015 06:48
asfgit pushed a commit that referenced this pull request Mar 13, 2015
If the cleaner is stopped, we shouldn't print a huge stack trace when the cleaner thread is interrupted because we purposefully did this.

Author: Andrew Or <[email protected]>

Closes #4882 from andrewor14/cleaner-interrupt and squashes the following commits:

8652120 [Andrew Or] Just a hot fix
asfgit pushed a commit that referenced this pull request Mar 22, 2015
If the cleaner is stopped, we shouldn't print a huge stack trace when the cleaner thread is interrupted because we purposefully did this.

Author: Andrew Or <[email protected]>

Closes #4882 from andrewor14/cleaner-interrupt and squashes the following commits:

8652120 [Andrew Or] Just a hot fix
asfgit pushed a commit that referenced this pull request Mar 22, 2015
If the cleaner is stopped, we shouldn't print a huge stack trace when the cleaner thread is interrupted because we purposefully did this.

Author: Andrew Or <[email protected]>

Closes #4882 from andrewor14/cleaner-interrupt and squashes the following commits:

8652120 [Andrew Or] Just a hot fix
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.

4 participants