[GitHub] spark pull request: [SPARK-7977] [Build] Disallowing println

2015-06-29 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/7093#discussion_r33504060 --- Diff: core/src/main/scala/org/apache/spark/deploy/Client.scala --- @@ -125,10 +126,12 @@ private class ClientActor(driverArgs: ClientArguments, conf:

[GitHub] spark pull request: [SPARK-2645] [Core] Allow SparkEnv.stop() to b...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6973#issuecomment-116814853 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

[GitHub] spark pull request: [SPARK-8704] [ML] [PySpark] Add additional met...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7086#issuecomment-116815350 Merged build finished. Test PASSed. --- 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: [SPARK-5989] [MLlib] Model save/load for LDA

2015-06-29 Thread MechCoder
Github user MechCoder commented on the pull request: https://github.com/apache/spark/pull/6948#issuecomment-116815596 Any other comments? @hhbyyh --- 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-8711] [ML] Add additional methods to Py...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7095#issuecomment-116823350 Merged build finished. Test PASSed. --- 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: [SPARK-8675] Executors created by LocalBackend...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7091#issuecomment-116828125 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-8579] [SQL] support arbitrary object in...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6959#issuecomment-116828143 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-6980] [CORE] Akka timeout exceptions in...

2015-06-29 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/6205#issuecomment-116833560 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 this

[GitHub] spark pull request: [SPARK-8372] Do not show applications that hav...

2015-06-29 Thread vanzin
GitHub user vanzin opened a pull request: https://github.com/apache/spark/pull/7097 [SPARK-8372] Do not show applications that haven't recorded their app ID yet. Showing these applications may lead to weird behavior in the History Server. For old logs, if the app ID is

[GitHub] spark pull request: [SPARK-6980] [CORE] Akka timeout exceptions in...

2015-06-29 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/6205#issuecomment-116833401 This never ran test. Jenkins, this is ok to test. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark pull request: [SPARK-8372] Do not show applications that hav...

2015-06-29 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/7097#issuecomment-116833772 I made some small changes on top of @carsonwang original patch, to account for old Spark versions. Most of my added changes are in the test code. --- If your project is

[GitHub] spark pull request: [SPARK-8372] Do not show applications that hav...

2015-06-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7097#issuecomment-116836938 [Test build #36032 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36032/consoleFull) for PR 7097 at commit

[GitHub] spark pull request: [SPARK-8710] [SQL] Change ScalaReflection.mirr...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7094#issuecomment-116841482 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-8710] [SQL] Change ScalaReflection.mirr...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7094#issuecomment-116841501 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

[GitHub] spark pull request: [SPARK-7810] [pyspark] solve python rdd socket...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6338#issuecomment-116841488 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-7810] [pyspark] solve python rdd socket...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6338#issuecomment-116841500 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

[GitHub] spark pull request: [SPARK-8579] [SQL] support arbitrary object in...

2015-06-29 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/6959#discussion_r33516110 --- Diff: sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeFixedWidthAggregationMap.java --- @@ -70,67 +92,38 @@ private

[GitHub] spark pull request: [SPARK-6910] [SQL] Support for pushing predica...

2015-06-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6921#issuecomment-116857231 [Test build #36026 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36026/console) for PR 6921 at commit

[GitHub] spark pull request: [SPARK-8579] [SQL] support arbitrary object in...

2015-06-29 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/6959#discussion_r33517761 --- Diff: sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/util/UniqueObjectPool.java --- @@ -0,0 +1,59 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request: [SPARK-6785][SQL] fix DateTimeUtils for dates ...

2015-06-29 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/6983#issuecomment-116863918 +1 to @squito 's suggestion, and I'll add that you can also force push if you do a rebase instead of a merge. Eitherway, github only is going to look at whatever the

[GitHub] spark pull request: [SPARK-6707] [CORE][MESOS]: Mesos Scheduler sh...

2015-06-29 Thread dragos
Github user dragos commented on the pull request: https://github.com/apache/spark/pull/5563#issuecomment-116874376 I still think this is a very useful feature and would love to have it in. We could look at [Marathon

[GitHub] spark pull request: [SPARK-8710] [SQL] Change ScalaReflection.mirr...

2015-06-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7094#issuecomment-116874339 [Test build #36037 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36037/console) for PR 7094 at commit

[GitHub] spark pull request: [HOTFIX] Fix whitespace style error

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7102#issuecomment-116874487 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-8713] Make codegen thread safe

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7101#issuecomment-116874493 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-8713] Make codegen thread safe

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7101#issuecomment-116874511 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

[GitHub] spark pull request: [HOTFIX] Fix whitespace style error

2015-06-29 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/7102#issuecomment-116874549 Merged. --- 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

[GitHub] spark pull request: [HOTFIX] Fix whitespace style error

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7102#issuecomment-116874507 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

[GitHub] spark pull request: [SPARK-8357] [SQL] Memory leakage on unsafe ag...

2015-06-29 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/6810#discussion_r33524575 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/AggregateSuite.scala --- @@ -0,0 +1,59 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-8690][SQL] Add a setting to disable Spa...

2015-06-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7070#issuecomment-116875664 [Test build #36048 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36048/consoleFull) for PR 7070 at commit

[GitHub] spark pull request: [SPARK-8713] Make codegen thread safe

2015-06-29 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/7101#issuecomment-116875730 LGTM. To make sure I understand, this still requires that the caller passes in a new set of expression for each thread in order to be safe? (which I think we get for

[GitHub] spark pull request: [SPARK-8019] [SPARKR] Support SparkR spawning ...

2015-06-29 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/6557#issuecomment-116875643 LGTM. We can document this later. --- 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-8711] [ML] Add additional methods to Py...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7095#issuecomment-116814806 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-7977] [Build] Disallowing println

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7093#issuecomment-116814818 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] spark pull request: [SPARK-8656][WebUI] Fix the webUI and JSON API...

2015-06-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7038#issuecomment-116834206 [Test build #36030 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36030/console) for PR 7038 at commit

[GitHub] spark pull request: [SPARK-8592] [CORE] CoarseGrainedExecutorBacke...

2015-06-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7077#issuecomment-116834302 [Test build #36028 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36028/consoleFull) for PR 7077 at commit

[GitHub] spark pull request: [SQL][Minor] Skip unresolved expression for In...

2015-06-29 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/6145#issuecomment-116834216 @scwf @cloud-fan Thanks! I'm merging this to branch-1.4. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark pull request: [SPARK-8615][Documentation]Fixed Sample deprec...

2015-06-29 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/7039#issuecomment-116834566 @yhuai --- 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

[GitHub] spark pull request: Spark-8660 - Convert JavaDoc style comments in...

2015-06-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7096#issuecomment-116834309 [Test build #36027 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36027/consoleFull) for PR 7096 at commit

[GitHub] spark pull request: [SPARK-8603] [sparkR] In windows, Incorrect fi...

2015-06-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7025#issuecomment-116838119 [Test build #36035 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36035/consoleFull) for PR 7025 at commit

[GitHub] spark pull request: [SPARK-8538][SPARK-8539][ML] Linear Regression...

2015-06-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7099#issuecomment-116837958 [Test build #36034 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36034/consoleFull) for PR 7099 at commit

[GitHub] spark pull request: SPARK-7171: Added a method to retrieve metrics...

2015-06-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5805#issuecomment-116841829 [Test build #36031 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36031/console) for PR 5805 at commit

[GitHub] spark pull request: [SPARK-7810] [pyspark] solve python rdd socket...

2015-06-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6338#issuecomment-116841847 [Test build #36038 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36038/consoleFull) for PR 6338 at commit

[GitHub] spark pull request: [SPARK-8710] [SQL] Change ScalaReflection.mirr...

2015-06-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7094#issuecomment-116841874 [Test build #36037 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36037/consoleFull) for PR 7094 at commit

[GitHub] spark pull request: SPARK-7171: Added a method to retrieve metrics...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5805#issuecomment-116842028 Merged build finished. Test PASSed. --- 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: [SPARK-8029][core] shuffleoutput per attempt

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6648#issuecomment-116844669 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-8029][core] shuffleoutput per attempt

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6648#issuecomment-116844688 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

[GitHub] spark pull request: [SPARK-2827][GraphX] Add collectDegreeDist to ...

2015-06-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4399#issuecomment-116846071 [Test build #36040 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36040/consoleFull) for PR 4399 at commit

[GitHub] spark pull request: [SPARK-6602][Core] Update Master, Worker, Clie...

2015-06-29 Thread aarondav
Github user aarondav commented on a diff in the pull request: https://github.com/apache/spark/pull/5392#discussion_r33513145 --- Diff: core/src/main/scala/org/apache/spark/deploy/Client.scala --- @@ -36,20 +36,30 @@ import org.apache.spark.util.{ActorLogReceive, AkkaUtils,

[GitHub] spark pull request: [SPARK-8019] [SPARKR] Support SparkR spawning ...

2015-06-29 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/6557#issuecomment-116846095 @davies @shivaram any updates on 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

[GitHub] spark pull request: [SPARK-4069] [YARN] When AppMaster finishes, t...

2015-06-29 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/5233#issuecomment-116847131 Looks like YARN-3415 is resolved. Is there still a fix that's needed on the Spark side @sryza? If not, shall we close this PR? --- If your project is set up for it,

[GitHub] spark pull request: [SPARK-6602][Core] Update Master, Worker, Clie...

2015-06-29 Thread aarondav
Github user aarondav commented on a diff in the pull request: https://github.com/apache/spark/pull/5392#discussion_r33513862 --- Diff: core/src/main/scala/org/apache/spark/deploy/Client.scala --- @@ -82,29 +92,38 @@ private class ClientActor(driverArgs: ClientArguments, conf:

[GitHub] spark pull request: [SPARK-6602][Core] Update Master, Worker, Clie...

2015-06-29 Thread aarondav
Github user aarondav commented on a diff in the pull request: https://github.com/apache/spark/pull/5392#discussion_r33513905 --- Diff: core/src/main/scala/org/apache/spark/deploy/Client.scala --- @@ -82,29 +92,38 @@ private class ClientActor(driverArgs: ClientArguments, conf:

[GitHub] spark pull request: [SPARK-8700][ML] Disable feature scaling in Lo...

2015-06-29 Thread holdenk
Github user holdenk commented on the pull request: https://github.com/apache/spark/pull/7080#issuecomment-116850826 Looks good to me with a few minor notes :) --- 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: [SPARK-6602][Core] Update Master, Worker, Clie...

2015-06-29 Thread aarondav
Github user aarondav commented on a diff in the pull request: https://github.com/apache/spark/pull/5392#discussion_r33515502 --- Diff: core/src/main/scala/org/apache/spark/deploy/client/AppClient.scala --- @@ -40,98 +37,139 @@ import org.apache.spark.util.{ActorLogReceive,

[GitHub] spark pull request: [SPARK-6707] [CORE][MESOS]: Mesos Scheduler sh...

2015-06-29 Thread ankurcha
Github user ankurcha commented on the pull request: https://github.com/apache/spark/pull/5563#issuecomment-116851005 @dragos That's a very good catch, I was going as per the http://mesos.apache.org/documentation/attributes-resources/ documentation page which says that the attribute

[GitHub] spark pull request: [SPARK-8598] [MLlib] Implementation of 1-sampl...

2015-06-29 Thread sryza
Github user sryza commented on a diff in the pull request: https://github.com/apache/spark/pull/6994#discussion_r33515526 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/stat/test/KSTest.scala --- @@ -0,0 +1,191 @@ +/* + * Licensed to the Apache Software Foundation

[GitHub] spark pull request: [SPARK-8434][SQL]Add a pretty parameter to t...

2015-06-29 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/6877#issuecomment-116855108 + 1 for truncate, otherwise LGTM. --- 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-7810] [pyspark] solve python rdd socket...

2015-06-29 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/6338#issuecomment-116854700 Merged into master, 1.4 and 1.3 branch --- 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

[GitHub] spark pull request: [CORE] [SPARK-6593] Provide option for HadoopR...

2015-06-29 Thread tigerquoll
Github user tigerquoll closed the pull request at: https://github.com/apache/spark/pull/5250 --- 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-6910] [SQL] Support for pushing predica...

2015-06-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6921#issuecomment-116861385 [Test build #36044 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36044/consoleFull) for PR 6921 at commit

[GitHub] spark pull request: [SPARK-6785][SQL] fix DateTimeUtils for dates ...

2015-06-29 Thread ckadner
Github user ckadner commented on the pull request: https://github.com/apache/spark/pull/6983#issuecomment-116861479 @marmbrus - Hi Michael, please let me know when you have time to review my pull-request. This PR no longer merges, so I would close this and create a new PR on top of

[GitHub] spark pull request: [SPARK-2645] [Core] Allow SparkEnv.stop() to b...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6973#issuecomment-116863246 Merged build finished. Test PASSed. --- 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: [SPARK-8579] [SQL] support arbitrary object in...

2015-06-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/6959 --- 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-7977] [Build] Disallowing println

2015-06-29 Thread jonalter
Github user jonalter commented on a diff in the pull request: https://github.com/apache/spark/pull/7093#discussion_r33524269 --- Diff: core/src/main/scala/org/apache/spark/deploy/Client.scala --- @@ -125,10 +126,12 @@ private class ClientActor(driverArgs: ClientArguments, conf:

[GitHub] spark pull request: [SPARK-8690][SQL] Add a setting to disable Spa...

2015-06-29 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/7070#issuecomment-116875110 ok to test --- 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

[GitHub] spark pull request: [SPARK-8581][SPARK-8584] Simplify checkpointin...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6968#issuecomment-116821437 Merged build finished. Test PASSed. --- 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: [SPARK-8710] [SQL] Change ScalaReflection.mirr...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7094#issuecomment-116820942 Merged build finished. Test FAILed. --- 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: [SPARK-8496] [TEST] Do not run slow tests for ...

2015-06-29 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/6917#discussion_r33507904 --- Diff: dev/run-tests.py --- @@ -639,32 +651,47 @@ def detect_binary_inop_with_mima(): run_cmd([os.path.join(SPARK_HOME, dev, mima)])

[GitHub] spark pull request: [SPARK-8675] Executors created by LocalBackend...

2015-06-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7091#issuecomment-116829532 [Test build #36024 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36024/console) for PR 7091 at commit

[GitHub] spark pull request: [SPARK-8070] [SQL] [PYSPARK] avoid spark jobs ...

2015-06-29 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/6606#issuecomment-116829717 Merging this. Thanks! --- 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-8660

2015-06-29 Thread Rosstin
GitHub user Rosstin opened a pull request: https://github.com/apache/spark/pull/7096 Spark-8660 Converted JavaDoc style comments in mllib/src/test/scala/org/apache/spark/ml/classification/LogisticRegressionSuite.scala to regular multiline comments, to make copy-pasting R commands

[GitHub] spark pull request: [SPARK-8675] Executors created by LocalBackend...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7091#issuecomment-116829730 Merged build finished. Test PASSed. --- 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: [SPARK-8669] [SQL] Fix crash with BINARY (ENUM...

2015-06-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7048#issuecomment-116833627 [Test build #36029 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36029/consoleFull) for PR 7048 at commit

[GitHub] spark pull request: Spark-8660 - Convert JavaDoc style comments in...

2015-06-29 Thread Rosstin
Github user Rosstin commented on the pull request: https://github.com/apache/spark/pull/7096#issuecomment-116834065 @rxin done, hope that's not too long --- 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

[GitHub] spark pull request: [SPARK-7977] [Build] Disallowing println

2015-06-29 Thread tdas
Github user tdas commented on a diff in the pull request: https://github.com/apache/spark/pull/7093#discussion_r33508854 --- Diff: streaming/src/test/scala/org/apache/spark/streaming/scheduler/JobGeneratorSuite.scala --- @@ -76,7 +76,9 @@ class JobGeneratorSuite extends

[GitHub] spark pull request: [SPARK-8619][Streaming]Don't recover keytab an...

2015-06-29 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/7008#discussion_r33510186 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/Checkpoint.scala --- @@ -44,11 +44,19 @@ class Checkpoint(@transient ssc: StreamingContext,

[GitHub] spark pull request: [SPARK-8538][SPARK-8539][ML] Linear Regression...

2015-06-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7099#issuecomment-116838806 [Test build #36034 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36034/console) for PR 7099 at commit

[GitHub] spark pull request: [Spark-8660] - Convert JavaDoc style comments ...

2015-06-29 Thread Rosstin
Github user Rosstin commented on the pull request: https://github.com/apache/spark/pull/7096#issuecomment-116838453 @rxin Of course, my bad --- 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-8393] [Streaming] Handle InterruptedExc...

2015-06-29 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/6969#issuecomment-116843259 @rekhajoshm can you address the comments? In particular, I think the proposal is just to fix the documentation for now. I would recommend that we close this PR and

[GitHub] spark pull request: [SPARK-8056][SQL] Design an easier way to cons...

2015-06-29 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/6686#issuecomment-116848080 Merging this in. Thanks @ilganeli! --- 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-6602][Core] Update Master, Worker, Clie...

2015-06-29 Thread aarondav
Github user aarondav commented on a diff in the pull request: https://github.com/apache/spark/pull/5392#discussion_r33514893 --- Diff: core/src/main/scala/org/apache/spark/deploy/client/AppClient.scala --- @@ -40,98 +37,139 @@ import org.apache.spark.util.{ActorLogReceive,

[GitHub] spark pull request: [SPARK-8603] [sparkR] In windows, Incorrect fi...

2015-06-29 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/7025#issuecomment-116849288 The code change looks ok but I need to test it on a windows machine to see what is going on. Will try to do this soon --- If your project is set up for it, you can

[GitHub] spark pull request: [SPARK-8660][ML] Convert JavaDoc style comment...

2015-06-29 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/7096#issuecomment-116856496 Thanks - merging this in 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 not

[GitHub] spark pull request: [SPARK-8660][ML] Convert JavaDoc style comment...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7096#issuecomment-116856432 Merged build finished. Test PASSed. --- 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: [SPARK-8592] [CORE] CoarseGrainedExecutorBacke...

2015-06-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7077#issuecomment-116859209 [Test build #36028 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36028/consoleFull) for PR 7077 at commit

[GitHub] spark pull request: [SPARK-8130] - Made spark.files.useFetchCache ...

2015-06-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7051#issuecomment-116860066 [Test build #36018 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36018/console) for PR 7051 at commit

[GitHub] spark pull request: [SPARK-8567] [SQL] Add logs to record the prog...

2015-06-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7092#issuecomment-116860321 [Test build #36019 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36019/consoleFull) for PR 7092 at commit

[GitHub] spark pull request: [SPARK-6602][Core] Update Master, Worker, Clie...

2015-06-29 Thread aarondav
Github user aarondav commented on a diff in the pull request: https://github.com/apache/spark/pull/5392#discussion_r33519086 --- Diff: core/src/main/scala/org/apache/spark/deploy/worker/Worker.scala --- @@ -258,41 +299,65 @@ private[worker] class Worker( } }

[GitHub] spark pull request: [SPARK-8130] - Made spark.files.useFetchCache ...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7051#issuecomment-116860168 Merged build finished. Test PASSed. --- 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: [SPARK-7977] [Build] Disallowing println

2015-06-29 Thread jonalter
Github user jonalter commented on a diff in the pull request: https://github.com/apache/spark/pull/7093#discussion_r33523678 --- Diff: core/src/main/scala/org/apache/spark/rdd/RDD.scala --- @@ -659,6 +660,7 @@ abstract class RDD[T: ClassTag]( * @param separateWorkingDir Use

[GitHub] spark pull request: [SPARK-8357] [SQL] Memory leakage on unsafe ag...

2015-06-29 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/6810#discussion_r33523617 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkPlan.scala --- @@ -153,13 +153,14 @@ abstract class SparkPlan extends

[GitHub] spark pull request: [SPARK-8715] ArrayOutOfBoundsException fixed f...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7100#issuecomment-116875481 Merged build finished. Test FAILed. --- 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: [SPARK-8233][SQL] misc function: hash

2015-06-29 Thread davies
Github user davies commented on a diff in the pull request: https://github.com/apache/spark/pull/6971#discussion_r33504350 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/misc.scala --- @@ -47,4 +48,44 @@ case class Md5(child: Expression)

[GitHub] spark pull request: [SPARK-8704] [ML] [PySpark] Add additional met...

2015-06-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7086#issuecomment-116815269 [Test build #36015 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36015/console) for PR 7086 at commit

[GitHub] spark pull request: [SPARK-8711] [ML] Add additional methods to Py...

2015-06-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7095#issuecomment-116816972 [Test build #36023 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36023/consoleFull) for PR 7095 at commit

[GitHub] spark pull request: [SPARK-8711] [ML] Add additional methods to Py...

2015-06-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7095#issuecomment-116820863 Merged build finished. Test FAILed. --- 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: [SPARK-8581][SPARK-8584] Simplify checkpointin...

2015-06-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6968#issuecomment-116821037 [Test build #36006 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36006/console) for PR 6968 at commit

[GitHub] spark pull request: [SPARK-8675] Executors created by LocalBackend...

2015-06-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7091#issuecomment-116828513 [Test build #36024 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36024/consoleFull) for PR 7091 at commit

[GitHub] spark pull request: [SPARK-8592] [CORE] CoarseGrainedExecutorBacke...

2015-06-29 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/7077#issuecomment-116832037 Wow, tests are really flaky today... --- 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

[GitHub] spark pull request: [SPARK-8592] [CORE] CoarseGrainedExecutorBacke...

2015-06-29 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/7077#issuecomment-116832055 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

<    1   2   3   4   5   6   7   8   9   10   >