[GitHub] spark pull request: SPARK-4628: Put all external projects behind a...

2014-11-26 Thread arahuja
Github user arahuja commented on a diff in the pull request: https://github.com/apache/spark/pull/3485#discussion_r20958627 --- Diff: pom.xml --- @@ -1201,6 +1196,18 @@ /dependencies /profile +!-- External projects are not built in less this flag

[GitHub] spark pull request: [SPARK-4344][DOCS] adding documentation on spa...

2014-11-21 Thread arahuja
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

[GitHub] spark pull request: [SPARK-4480] Avoid many small spills in extern...

2014-11-19 Thread arahuja
Github user arahuja commented on the pull request: https://github.com/apache/spark/pull/3353#issuecomment-63700762 Was this not going into master? --- 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

[GitHub] spark pull request: [SPARK-4344][DOCS] adding documentation on spa...

2014-11-18 Thread arahuja
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

[GitHub] spark pull request: [SPARK-4281][Build] Package Yarn shuffle servi...

2014-11-12 Thread arahuja
Github user arahuja commented on the pull request: https://github.com/apache/spark/pull/3147#issuecomment-62834492 I had that as well when I did provide -Pyarn to `make-distribution.sh` --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-4344][DOCS] adding documentation on spa...

2014-11-11 Thread arahuja
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] spark pull request: [SPARK-4344][DOCS] adding documentation on spa...

2014-11-11 Thread arahuja
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] spark pull request: [SPARK-4344][DOCS] adding documentation on spa...

2014-11-11 Thread arahuja
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] spark pull request: [SPARK-2878]: Fix custom spark.kryo.registrato...

2014-08-17 Thread arahuja
Github user arahuja commented on the pull request: https://github.com/apache/spark/pull/1890#issuecomment-52446389 I think this may be the issue I have been wrangling with the last couple days. I see a variety of odd Kryo related errors, slightly different each time