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.

Sign up for GitHub

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

Jump to bottom

[SPARK-5831][Streaming]When checkpoint file size is bigger than 10, then delete the old ones #4621

Closed
wants to merge 1 commit into from
Closed

[SPARK-5831][Streaming]When checkpoint file size is bigger than 10, then delete the old ones #4621

wants to merge 1 commit into from
+1 −1

Conversation

XuTingjun
Copy link
Contributor

No description provided.

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@srowen
Copy link
Member

srowen commented Feb 16, 2015

EDIT: oh right, it's obvious if I actually look at the next line. LGTM.

@srowen
Copy link
Member

srowen commented Feb 16, 2015

ok to test

@SparkQA
Copy link

SparkQA commented Feb 16, 2015

Test build #27552 has started for PR 4621 at commit b5f2816.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Feb 16, 2015

Test build #27552 has finished for PR 4621 at commit b5f2816.

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

@AmplabJenkins
Copy link

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

@srowen
Copy link
Member

srowen commented Feb 16, 2015

Retest this please

@SparkQA
Copy link

SparkQA commented Feb 16, 2015

Test build #27554 has started for PR 4621 at commit b5f2816.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Feb 16, 2015

Test build #27554 has finished for PR 4621 at commit b5f2816.

  • 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/27554/
Test PASSed.

@srowen
Copy link
Member

srowen commented Feb 16, 2015

It doesn't actually affect anything, since take(n) is empty for any n <= 0, but this looks like it was clearly a typo.

@asfgit asfgit closed this in 1115e8e Feb 16, 2015
@XuTingjun XuTingjun deleted the checkpointFile branch February 17, 2015 01:33
zzcclp pushed a commit to zzcclp/spark that referenced this pull request Mar 11, 2015
@XuTingjun @zzcclp
[SPARK-5831][Streaming]When checkpoint file size is bigger than 10, t…
6a4f909 
…hen delete the old ones

Author: Xutingjun <[email protected]>

Closes apache#4621 from XuTingjun/checkpointFile and squashes the following commits:

b5f2816 [Xutingjun] fix bug
zzcclp pushed a commit to zzcclp/spark that referenced this pull request Mar 12, 2015
@XuTingjun @zzcclp
[SPARK-5831][Streaming]When checkpoint file size is bigger than 10, t…
3080a12 
…hen delete the old ones

Author: Xutingjun <[email protected]>

Closes apache#4621 from XuTingjun/checkpointFile and squashes the following commits:

b5f2816 [Xutingjun] fix bug
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@XuTingjun @AmplabJenkins @srowen @SparkQA