Github user tgravescs commented on the pull request:
https://github.com/apache/spark/pull/3209#issuecomment-64405811
test this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user tgravescs commented on the pull request:
https://github.com/apache/spark/pull/3209#issuecomment-64405912
@vanzin I think I'll pull this in and you will have to remove it again in
https://github.com/apache/spark/pull/3233
---
If your project is set up for it, you can
Github user tgravescs commented on the pull request:
https://github.com/apache/spark/pull/3209#issuecomment-64406285
I pulled this into both master and branch-1.2
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/3209
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user arahuja commented on the pull request:
https://github.com/apache/spark/pull/3209#issuecomment-64003170
@vanzin that sounds reasonable, though confusing if #3233 does not go in
soon, but anyways, sounds fine to me. Is there something I should do for that?
Reopen this
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/3209#issuecomment-64009711
I don't know if it's possible to move a PR to a different branch (or
whether you need to create a new one). In any case, it's not a big deal if this
goes into master.
Github user arahuja commented on the pull request:
https://github.com/apache/spark/pull/3209#issuecomment-63507703
Updated it to the one line change on userClassPathFirst, but just as happy
to close it if #3233 goes in
---
If your project is set up for it, you can reply to this
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/3209#issuecomment-63518490
I'm ok with the message, although if you only add it on branch-1.2 it would
save me the work of having to delete it as part of my other PR. :-)
---
If your project is
Github user tgravescs commented on the pull request:
https://github.com/apache/spark/pull/3209#issuecomment-63078583
so @vanzin put up a pull request to implement it on yarn
(https://github.com/apache/spark/pull/3233).
Can we just close this one in favor of that? I'm fine
GitHub user arahuja opened a pull request:
https://github.com/apache/spark/pull/3209
[SPARK-4344][DOCS] adding documentation on spark.yarn.user.classpath.first
The documentation for the two parameters is the same with a pointer from
the standalone parameter to the yarn parameter
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/3209#issuecomment-62590575
Can one of the admins verify this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/3209#discussion_r20172562
--- Diff: docs/configuration.md ---
@@ -224,6 +224,7 @@ Apart from these, the following properties are also
available, and may be useful
Github user arahuja commented on a diff in the pull request:
https://github.com/apache/spark/pull/3209#discussion_r20178399
--- Diff: docs/configuration.md ---
@@ -224,6 +224,7 @@ Apart from these, the following properties are also
available, and may be useful
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/3209#discussion_r20181813
--- Diff: docs/configuration.md ---
@@ -224,6 +224,7 @@ Apart from these, the following properties are also
available, and may be useful
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/3209#discussion_r20182510
--- Diff: docs/configuration.md ---
@@ -224,6 +224,7 @@ Apart from these, the following properties are also
available, and may be useful
Github user arahuja commented on a diff in the pull request:
https://github.com/apache/spark/pull/3209#discussion_r20183100
--- Diff: docs/configuration.md ---
@@ -224,6 +224,7 @@ Apart from these, the following properties are also
available, and may be useful
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/3209#discussion_r20186306
--- Diff: docs/configuration.md ---
@@ -224,6 +224,7 @@ Apart from these, the following properties are also
available, and may be useful
17 matches
Mail list logo
