[GitHub] spark pull request: delete no use var

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/613#issuecomment-41993038 Merged build finished. --- 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

[GitHub] spark pull request: SPARK-1695: java8-tests compiler error: packag...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/611#issuecomment-41993037 Merged build finished. All automated tests passed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] spark pull request: SPARK-1676: (branch-0.9 fix) Cache Hadoop UGIs...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/618#issuecomment-41993044 Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/14618/ --- If your project is set up for it, you can

[GitHub] spark pull request: delete no use var

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/613#issuecomment-41993042 Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/14615/ --- If your project is set up for it, you can

[GitHub] spark pull request: SPARK-1676: (branch-0.9 fix) Cache Hadoop UGIs...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/618#issuecomment-41993040 Merged build finished. --- 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

[GitHub] spark pull request: SPARK-1695: java8-tests compiler error: packag...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/611#issuecomment-41993041 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/14617/ --- If your project

[GitHub] spark pull request: [SQL] Whitelist Hive Tests

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/596#issuecomment-41993039 Merged build finished. All automated tests passed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] spark pull request: [SQL] Whitelist Hive Tests

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/596#issuecomment-41993043 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/14616/ --- If your project

[GitHub] spark pull request: [SQL] Better logging when applying rules.

2014-05-02 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/616#issuecomment-41993427 Jenkins, retest 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

[GitHub] spark pull request: delete no use var

2014-05-02 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/613#issuecomment-41993403 Jenkins, 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] spark pull request: delete no use var

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/613#issuecomment-41993660 Merged build triggered. --- 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

[GitHub] spark pull request: SPARK-1676: (branch-0.9 fix) Cache Hadoop UGIs...

2014-05-02 Thread aarondav
Github user aarondav commented on the pull request: https://github.com/apache/spark/pull/618#issuecomment-41993672 Jenkins, retest 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

[GitHub] spark pull request: [SQL] Better logging when applying rules.

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/616#issuecomment-41993659 Merged build triggered. --- 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

[GitHub] spark pull request: [SQL] Better logging when applying rules.

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/616#issuecomment-41993667 Merged build started. --- 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

[GitHub] spark pull request: SPARK-1676: (branch-0.9 fix) Cache Hadoop UGIs...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/618#issuecomment-41994045 Merged build triggered. --- 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

[GitHub] spark pull request: SPARK-1676: (branch-0.9 fix) Cache Hadoop UGIs...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/618#issuecomment-41994055 Merged build started. --- 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

[GitHub] spark pull request: SPARK-1676: (branch-0.9 fix) Cache Hadoop UGIs...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/618#issuecomment-41998357 Merged build finished. All automated tests passed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] spark pull request: [SQL] Better logging when applying rules.

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/616#issuecomment-41998359 Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/14619/ --- If your project is set up for it, you can

[GitHub] spark pull request: delete no use var

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/613#issuecomment-41998358 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/14620/ --- If your project

[GitHub] spark pull request: delete no use var

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/613#issuecomment-41998355 Merged build finished. All automated tests passed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] spark pull request: [SQL] Better logging when applying rules.

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/616#issuecomment-41998356 Merged build finished. --- 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

[GitHub] spark pull request: fix Apache License for join.py

2014-05-02 Thread kanzhang
Github user kanzhang closed the pull request at: https://github.com/apache/spark/pull/592 --- 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] spark pull request: SPARK-1668: Add implicit preference as an opti...

2014-05-02 Thread techaddict
Github user techaddict commented on the pull request: https://github.com/apache/spark/pull/597#issuecomment-42003703 Mapping rating in case of ImplicitPref to `{r=0 -- 0`, `r0 -- 1}`, `Rating(fields(0).toInt, fields(1).toInt, fields(2).toDouble)` to `Rating(fields(0).toInt,

[GitHub] spark pull request: get sparkhome from sparkconf Preferentially

2014-05-02 Thread scwf
Github user scwf commented on a diff in the pull request: https://github.com/apache/spark/pull/615#discussion_r1831 --- Diff: core/src/main/scala/org/apache/spark/deploy/worker/Worker.scala --- @@ -83,7 +83,8 @@ private[spark] class Worker( @volatile var registered =

[GitHub] spark pull request: get sparkhome from sparkconf Preferentially

2014-05-02 Thread pwendell
Github user pwendell commented on a diff in the pull request: https://github.com/apache/spark/pull/615#discussion_r12223079 --- Diff: core/src/main/scala/org/apache/spark/deploy/worker/Worker.scala --- @@ -83,7 +83,8 @@ private[spark] class Worker( @volatile var registered

[GitHub] spark pull request: [WIP] SPARK-1698: Improve spark integration

2014-05-02 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/598#issuecomment-42016578 Can you explain why this resolves any of the issues you identified? See my comment son SPARK-1698. shade is generally better than assembly for making uber jars, not least

[GitHub] spark pull request: SPARK-1556: bump jets3t version to 0.9.0

2014-05-02 Thread darose
Github user darose commented on the pull request: https://github.com/apache/spark/pull/468#issuecomment-42027309 Man oh man, I cannot get this to work no way no how. I tried rebuilding spark using the jets3t 0.9 jar, then tried rebuilding shark doing the same. I keep getting a

[GitHub] spark pull request: SPARK-1565, update examples to be used with sp...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/552#issuecomment-42027968 Build triggered. --- 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

[GitHub] spark pull request: SPARK-1565, update examples to be used with sp...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/552#issuecomment-42027982 Build started. --- 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] spark pull request: SPARK-1565, update examples to be used with sp...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/552#issuecomment-42028160 Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/14622/ --- If your project is set up for it, you can

[GitHub] spark pull request: SPARK-1565, update examples to be used with sp...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/552#issuecomment-42028463 Build started. --- 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] spark pull request: Reset the thread classLoader in the Mesos Exec...

2014-05-02 Thread bouk
GitHub user bouk opened a pull request: https://github.com/apache/spark/pull/620 Reset the thread classLoader in the Mesos Executor This is because Mesos calls it with a different environment or something, the result is that the Spark jar is missing and it can't load classes.

[GitHub] spark pull request: Reset the thread classLoader in the Mesos Exec...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/620#issuecomment-42070660 Merged build triggered. --- 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

[GitHub] spark pull request: Reset the thread classLoader in the Mesos Exec...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/620#issuecomment-42070992 Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/14624/ --- If your project is set up for it, you can

[GitHub] spark pull request: SPARK-1695: java8-tests compiler error: packag...

2014-05-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/611 --- 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] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread tgravescs
GitHub user tgravescs opened a pull request: https://github.com/apache/spark/pull/621 [WIP] SPARK-1676: Cache Hadoop UGIs by default to prevent FileSystem leak Move the doAs in Executor higher up so that we only have 1 ugi and aren't leaking filesystems. Fix spark on yarn to

[GitHub] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/621#issuecomment-42077867 Merged build triggered. --- 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

[GitHub] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/621#issuecomment-42077873 Merged build started. --- 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

[GitHub] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/621#issuecomment-42078001 Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/14625/ --- If your project is set up for it, you can

[GitHub] spark pull request: [SPARK-1620] Handle uncaught exceptions in fun...

2014-05-02 Thread markhamstra
GitHub user markhamstra opened a pull request: https://github.com/apache/spark/pull/622 [SPARK-1620] Handle uncaught exceptions in function run by Akka scheduler If the intended behavior was that uncaught exceptions thrown in functions being run by the Akka scheduler would end up

[GitHub] spark pull request: [SPARK-1620] Handle uncaught exceptions in fun...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/622#issuecomment-42078735 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-1620] Handle uncaught exceptions in fun...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/622#issuecomment-42078743 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-1620] Handle uncaught exceptions in fun...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/622#issuecomment-42078848 Merged build finished. --- 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

[GitHub] spark pull request: [SPARK-1620] Handle uncaught exceptions in fun...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/622#issuecomment-42078849 Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/14626/ --- If your project is set up for it, you can

[GitHub] spark pull request: [SPARK-1620] Handle uncaught exceptions in fun...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/622#issuecomment-42079991 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-1620] Handle uncaught exceptions in fun...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/622#issuecomment-4208 Merged build started. --- 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

[GitHub] spark pull request: Reset the thread classLoader in the Mesos Exec...

2014-05-02 Thread airhorns
Github user airhorns commented on the pull request: https://github.com/apache/spark/pull/620#issuecomment-42080286 @bouk the spark contribution guidelines ask you make a JIRA https://cwiki.apache.org/confluence/display/SPARK/Contributing+to+Spark --- If your project is set up for

[GitHub] spark pull request: Reset the thread classLoader in the Mesos Exec...

2014-05-02 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/620#issuecomment-42080589 Would you mind printing the exception that happens when you don't have this fix? I'm wondering which instantiation fails. In theory I don't think we should have to do

[GitHub] spark pull request: SPARK-1676: Cache Hadoop UGIs by default to pr...

2014-05-02 Thread aarondav
Github user aarondav commented on the pull request: https://github.com/apache/spark/pull/607#issuecomment-42081936 Ah, your point about making sure we clean up if we don't cache the UGIs is definitely correct, no reason not to do that. For your last question, you might note

[GitHub] spark pull request: [SPARK-1620] Handle uncaught exceptions in fun...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/622#issuecomment-42083578 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/14627/ --- If your project

[GitHub] spark pull request: [SPARK-1620] Handle uncaught exceptions in fun...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/622#issuecomment-42083577 Merged build finished. All automated tests passed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] spark pull request: SPARK-1004. PySpark on YARN

2014-05-02 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/30#issuecomment-42084965 Sure. In the event that we don't figure out what the issue is by the time we release a voting candidate for 1.0, we should at the very least document the peculiarity

[GitHub] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread aarondav
Github user aarondav commented on the pull request: https://github.com/apache/spark/pull/621#issuecomment-42086088 This seems pretty reasonable to me, but it assumes that there is no value in recreating the user and re-transferring the current user's credentials. Is this the case?

[GitHub] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/621#discussion_r12251357 --- Diff: core/src/main/scala/org/apache/spark/executor/MesosExecutorBackend.scala --- @@ -95,9 +95,13 @@ private[spark] class MesosExecutorBackend

[GitHub] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread aarondav
Github user aarondav commented on a diff in the pull request: https://github.com/apache/spark/pull/621#discussion_r12251478 --- Diff: core/src/main/scala/org/apache/spark/executor/CoarseGrainedExecutorBackend.scala --- @@ -94,25 +95,32 @@ private[spark] class

[GitHub] spark pull request: SPARK-1676: Cache Hadoop UGIs by default to pr...

2014-05-02 Thread tgravescs
Github user tgravescs commented on the pull request: https://github.com/apache/spark/pull/607#issuecomment-42087138 What is your concern with moving it up into the ExecutorBackend? I'd be ok with using the cache and closeAll (if config off) if the consensus is that moving it

[GitHub] spark pull request: Reset the thread classLoader in the Mesos Exec...

2014-05-02 Thread bouk
Github user bouk commented on the pull request: https://github.com/apache/spark/pull/620#issuecomment-42088169 Mesos `stderr` without this patch on master ``` WARNING: Logging before InitGoogleLogging() is written to STDERR I0502 17:31:42.672224 14688 exec.cpp:131]

[GitHub] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/621#discussion_r12252251 --- Diff: core/src/main/scala/org/apache/spark/executor/CoarseGrainedExecutorBackend.scala --- @@ -94,25 +95,32 @@ private[spark] class

[GitHub] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread aarondav
Github user aarondav commented on a diff in the pull request: https://github.com/apache/spark/pull/621#discussion_r12252315 --- Diff: core/src/main/scala/org/apache/spark/deploy/SparkHadoopUtil.scala --- @@ -24,25 +24,30 @@ import org.apache.hadoop.mapred.JobConf import

[GitHub] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread aarondav
Github user aarondav commented on a diff in the pull request: https://github.com/apache/spark/pull/621#discussion_r12252364 --- Diff: core/src/main/scala/org/apache/spark/executor/CoarseGrainedExecutorBackend.scala --- @@ -94,25 +95,32 @@ private[spark] class

[GitHub] spark pull request: Reset the thread classLoader in the Mesos Exec...

2014-05-02 Thread bouk
Github user bouk commented on the pull request: https://github.com/apache/spark/pull/620#issuecomment-42088660 Note that exchanging the line https://github.com/apache/spark/blob/e53eb4f0159ebd4d72c4bbc3586fdfc66ccacab7/core/src/main/scala/org/apache/spark/SparkEnv.scala#L165 with

[GitHub] spark pull request: Improved build configuration � �

2014-05-02 Thread tgravescs
Github user tgravescs commented on the pull request: https://github.com/apache/spark/pull/590#issuecomment-42089041 I'm getting errors trying to build this against yarn-alpha hadoop-0.23. Its a huge error message, but here is part of it: [ERROR] while

[GitHub] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread aarondav
Github user aarondav commented on the pull request: https://github.com/apache/spark/pull/621#issuecomment-42091219 I have tested this on standalone mode and confirmed that the file handles do not leak. --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark pull request: SPARK-1700: Close socket file descriptors on t...

2014-05-02 Thread aarondav
GitHub user aarondav opened a pull request: https://github.com/apache/spark/pull/623 SPARK-1700: Close socket file descriptors on task completion This will ensure that sockets do not build up over the course of a job, and that cancellation successfully cleans up sockets.

[GitHub] spark pull request: SPARK-1700: Close socket file descriptors on t...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/623#issuecomment-42091550 Merged build triggered. --- 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

[GitHub] spark pull request: SPARK-1700: Close socket file descriptors on t...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/623#issuecomment-42091554 Merged build started. --- 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

[GitHub] spark pull request: SPARK-1700: Close socket file descriptors on t...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/623#issuecomment-42092586 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/14628/ --- If your project

[GitHub] spark pull request: SPARK-1700: Close socket file descriptors on t...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/623#issuecomment-42092585 Merged build finished. All automated tests passed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread tgravescs
Github user tgravescs commented on the pull request: https://github.com/apache/spark/pull/621#issuecomment-42092592 There is no reason to recreate the user and repopulate the credentials/token unless the credentials/tokens are being updated in the ExecutorBackend process. On yarn

[GitHub] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread sryza
Github user sryza commented on the pull request: https://github.com/apache/spark/pull/621#issuecomment-42092767 To add to what Tom said, there's a distinction between renewing tokens and repopulating them. Renewing means extending the lifespan of existing tokens. Repopulating is

[GitHub] spark pull request: Reset the thread classLoader in the Mesos Exec...

2014-05-02 Thread dkanoafry
Github user dkanoafry commented on the pull request: https://github.com/apache/spark/pull/620#issuecomment-42093656 fwiw, I experienced the same error with branch-1.0 and mesos 0.17, and applying the patch in this PR fixes the problem. --- If your project is set up for it, you can

[GitHub] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/621#issuecomment-42094097 Merged build triggered. --- 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

[GitHub] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/621#issuecomment-42094099 Merged build started. --- 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

[GitHub] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/621#issuecomment-42094124 Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/14629/ --- If your project is set up for it, you can

[GitHub] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/621#issuecomment-42094123 Merged build finished. --- 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

[GitHub] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/621#issuecomment-42094276 Merged build started. --- 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

[GitHub] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/621#issuecomment-42094271 Merged build triggered. --- 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

[GitHub] spark pull request: Improved build configuration � �

2014-05-02 Thread witgo
Github user witgo commented on the pull request: https://github.com/apache/spark/pull/590#issuecomment-42094359 @tgravescs I tested many times, These are all can pass. `mvn clean package -DskipTests -Pyarn-alpha -Dhadoop.version=0.23.7 -Phadoop-0.23` `mvn clean package

[GitHub] spark pull request: Improved build configuration � �

2014-05-02 Thread tgravescs
Github user tgravescs commented on the pull request: https://github.com/apache/spark/pull/590#issuecomment-42094808 Sorry, looks like I missed that the build command changed to need -Phadoop-0.23. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/621#issuecomment-42094847 Merged build finished. --- 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

[GitHub] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/621#issuecomment-42094848 Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/14630/ --- If your project is set up for it, you can

[GitHub] spark pull request: Reset the thread classLoader in the Mesos Exec...

2014-05-02 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/620#issuecomment-42095686 Would you mind trying an alternative fix? ``` val cls = Class.forName(name, true, Utils.getContextOrSparkClassLoader) ``` We recently did

[GitHub] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/621#issuecomment-42095752 Jenkins, retest 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

[GitHub] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/621#issuecomment-42095785 Merged build triggered. --- 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

[GitHub] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/621#issuecomment-42095787 Merged build started. --- 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

[GitHub] spark pull request: SPARK-1556: bump jets3t version to 0.9.0

2014-05-02 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/468#issuecomment-42096004 @srowen I'd prefer not to remove it from the dependency graph if possible because it will break local builds. The best solution I see is to add a profile for Hadoop 2.3

[GitHub] spark pull request: delete no use var

2014-05-02 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/613#issuecomment-42096048 Thanks I merged this. --- 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] spark pull request: SPARK-1492. Update Spark YARN docs to use spar...

2014-05-02 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/601#issuecomment-42096177 Thanks Sandy, I've merged this. --- 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

[GitHub] spark pull request: SPARK-1556: bump jets3t version to 0.9.0

2014-05-02 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/468#issuecomment-42096201 @srowen if you'd like to take a crack at this by the way, please do. I'll probably look at it on Sunday if no one else has. --- If your project is set up for it, you

[GitHub] spark pull request: [SPARK-1681] Include datanucleus jars in Spark...

2014-05-02 Thread pwendell
Github user pwendell commented on a diff in the pull request: https://github.com/apache/spark/pull/610#discussion_r12254725 --- Diff: bin/compute-classpath.sh --- @@ -63,14 +64,21 @@ fi # built with Hive, so first check if the datanucleus jars exist, and then ensure the

[GitHub] spark pull request: [WIP] SPARK-1676: Cache Hadoop UGIs by default...

2014-05-02 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/621#issuecomment-42096413 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/14631/ --- If your project

[GitHub] spark pull request: SPARK-1700: Close socket file descriptors on t...

2014-05-02 Thread pwendell
Github user pwendell commented on a diff in the pull request: https://github.com/apache/spark/pull/623#discussion_r12254807 --- Diff: core/src/main/scala/org/apache/spark/api/python/PythonRDD.scala --- @@ -54,7 +54,16 @@ private[spark] class PythonRDD[T: ClassTag]( override

[GitHub] spark pull request: SPARK-1700: Close socket file descriptors on t...

2014-05-02 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/623#issuecomment-42096479 LGTM - were you able to test this in some way? --- 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] spark pull request: delete no use var

2014-05-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/613 --- 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] spark pull request: SPARK-1492. Update Spark YARN docs to use spar...

2014-05-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/601 --- 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