Skip to content

Conversation

jongyoul
Copy link
Member

...n mesos cluster mode

  • change master newer

…y in mesos cluster mode

- change master newer
@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@jongyoul
Copy link
Member Author

@mateiz @tgravescs @tnachen @andrewor14 @timothysc I fixed the older master issue by getting new forking sources. Test is again please.

@jongyoul
Copy link
Member Author

This is a new version of PR #2126 .

@timothysc
Copy link

+1
@tnachen
@mateiz

@andrewor14
Copy link
Contributor

add to whitelist

@SparkQA
Copy link

SparkQA commented Oct 31, 2014

Test build #22633 has started for PR 3034 at commit 42b2ed3.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Oct 31, 2014

Test build #22633 has finished for PR 3034 at commit 42b2ed3.

  • 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/22633/
Test FAILed.

@jongyoul
Copy link
Member Author

jongyoul commented Nov 1, 2014

@andrewor14 Is there any way to test a patch from my local machine? And check the test result, I don't know a specific problem about testing.

@andrewor14
Copy link
Contributor

It failed because of a Yarn compilation error that is now fixed. Let's try again. Jenkin, retest this please.

@SparkQA
Copy link

SparkQA commented Nov 1, 2014

Test build #22676 has started for PR 3034 at commit 42b2ed3.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Nov 1, 2014

Test build #22676 has finished for PR 3034 at commit 42b2ed3.

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

@jongyoul
Copy link
Member Author

jongyoul commented Nov 4, 2014

@andrewor14 Jenkins test succeed. Check and merge it please.

@timothysc
Copy link

+1.
Thanks for all your persistence and patience @jongyoul for seeing this through.

@andrewor14
Copy link
Contributor

Ok I'm merging this into master 1.2 and 1.1, thanks @jongyoul.

@asfgit asfgit closed this in f7ac8c2 Nov 5, 2014
asfgit pushed a commit that referenced this pull request Nov 5, 2014
…y i...

...n mesos cluster mode

- change master newer

Author: Jongyoul Lee <[email protected]>

Closes #3034 from jongyoul/SPARK-3223 and squashes the following commits:

42b2ed3 [Jongyoul Lee] SPARK-3223 runAsSparkUser cannot change HDFS write permission properly in mesos cluster mode - change master newer

(cherry picked from commit f7ac8c2)
Signed-off-by: Andrew Or <[email protected]>
asfgit pushed a commit that referenced this pull request Nov 5, 2014
…y i...

...n mesos cluster mode

- change master newer

Author: Jongyoul Lee <[email protected]>

Closes #3034 from jongyoul/SPARK-3223 and squashes the following commits:

42b2ed3 [Jongyoul Lee] SPARK-3223 runAsSparkUser cannot change HDFS write permission properly in mesos cluster mode - change master newer

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants