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.

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

SPARK-3223 runAsSparkUser cannot change HDFS write permission properly i... #3034

Closed
wants to merge 1 commit into from
Closed

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 <jongyoul@gmail.com>

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 <andrew@databricks.com>
asfgit pushed a commit that referenced this pull request Nov 5, 2014
…y i...

...n mesos cluster mode

- change master newer

Author: Jongyoul Lee <jongyoul@gmail.com>

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 <andrew@databricks.com>
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