Skip to content

[SPARK-4446] [SPARK CORE] #3306

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
Closed
wants to merge 1 commit into from
Closed

[SPARK-4446] [SPARK CORE] #3306

wants to merge 1 commit into from

Conversation

@Leolh
Copy link

@Leolh Leolh commented Nov 17, 2014

MetadataCleaner schedule task with a wrong param for delay time .

Update MetadataCleaner.scala
4a21f4e 
Fix a little mistake about delaySeconds .
@AmplabJenkins
Copy link

AmplabJenkins commented Nov 17, 2014

Can one of the admins verify this patch?

@andrewor14
Copy link
Contributor

andrewor14 commented Nov 19, 2014

ok to test. I think this is correct. @tdas?

@SparkQA
Copy link

SparkQA commented Nov 19, 2014

Test build #23626 has started for PR 3306 at commit 4a21f4e.

  • This patch merges cleanly.

@tdas
Copy link
Contributor

tdas commented Nov 19, 2014

Good catch! LGTM.

@andrewor14
Copy link
Contributor

andrewor14 commented Nov 19, 2014

Hey @Leolh can you rebase to master now that the MiMa thing is fixed? The currently running tests will fail.

@SparkQA
Copy link

SparkQA commented Nov 19, 2014

Test build #23626 has finished for PR 3306 at commit 4a21f4e.

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

@AmplabJenkins
Copy link

AmplabJenkins commented Nov 19, 2014

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

@vanzin
Copy link
Contributor

vanzin commented Nov 19, 2014

Also, could you also fill in the title for the PR with something else aside from the bug number? Thanks!

@andrewor14
Copy link
Contributor

andrewor14 commented Nov 20, 2014

retest this please

@SparkQA
Copy link

SparkQA commented Nov 20, 2014

Test build #23644 has started for PR 3306 at commit 4a21f4e.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Nov 20, 2014

Test build #23644 has finished for PR 3306 at commit 4a21f4e.

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

@AmplabJenkins
Copy link

AmplabJenkins commented Nov 20, 2014

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

@andrewor14
Copy link
Contributor

andrewor14 commented Nov 20, 2014

Ok merging into 1.2 and master

@asfgit asfgit closed this in e216ffa Nov 20, 2014
asfgit pushed a commit that referenced this pull request Nov 20, 2014
[SPARK-4446] [SPARK CORE]
83d24ef 
MetadataCleaner schedule task with a wrong param for delay time .

Author: Leolh <[email protected]>

Closes #3306 from Leolh/master and squashes the following commits:

4a21f4e [Leolh] Update MetadataCleaner.scala

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

6 participants

@Leolh @AmplabJenkins @andrewor14 @SparkQA @tdas @vanzin