[GitHub] spark pull request: SPARK-5841 [CORE] [HOTFIX 2] Memory leak in Di...

2015-02-19 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/4690#issuecomment-75016363 No, this was already fixed in https://github.com/apache/spark/commit/49c19fdbad57f0609bbcc9278f9eaa8115a73604 I don't think you have that commit yet here. The problem is

[GitHub] spark pull request: [SPARK-5881][SQL] RDD remains cached after the...

2015-02-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4689#issuecomment-75015004 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5881][SQL] RDD remains cached after the...

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

[GitHub] spark pull request: [SQL] Checking data types when resolving types

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

[GitHub] spark pull request: SPARK-5841 [CORE] [HOTFIX 2] Memory leak in Di...

2015-02-19 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/4690#issuecomment-75019602 Ah right. I was glancing at the number of commits and thought it hadn't been rebased. The problem isn't this shutdown hook but another one in `yarn.ApplicationMaster`.

[GitHub] spark pull request: SPARK-5841 [CORE] [HOTFIX 2] Memory leak in Di...

2015-02-19 Thread nishkamravi2
Github user nishkamravi2 commented on the pull request: https://github.com/apache/spark/pull/4690#issuecomment-75020545 Agreed. That's why a try-catch block seems enough for this one. Ok with removing logging. --- If your project is set up for it, you can reply to this email and

[GitHub] spark pull request: SPARK-3172 and SPARK-3577

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

[GitHub] spark pull request: Spark-5708: Add Slf4jSink to Spark Metrics

2015-02-19 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/4644#issuecomment-75017213 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-5436] [MLlib] Validate GradientBoostedT...

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

[GitHub] spark pull request: [SPARK-5436] [MLlib] Validate GradientBoostedT...

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

[GitHub] spark pull request: [SPARK-5436] [MLlib] Validate GradientBoostedT...

2015-02-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4677#issuecomment-75017712 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: Spark-5708: Add Slf4jSink to Spark Metrics

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

[GitHub] spark pull request: [SPARK-5904][SQL] DataFrame API fixes.

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

[GitHub] spark pull request: [SPARK-5741][SQL] Support the path contains co...

2015-02-19 Thread watermen
Github user watermen commented on the pull request: https://github.com/apache/spark/pull/4532#issuecomment-75016913 @yhuai Can you review it? --- 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-5436] [MLlib] Validate GradientBoostedT...

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

[GitHub] spark pull request: [SPARK-5436] [MLlib] Validate GradientBoostedT...

2015-02-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4677#issuecomment-75016842 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: SPARK-3172 and SPARK-3577

2015-02-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/2504#issuecomment-75015886 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SQL] Checking data types when resolving types

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

[GitHub] spark pull request: [SQL] Checking data types when resolving types

2015-02-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4685#issuecomment-75018748 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: SPARK-5841 [CORE] [HOTFIX 2] Memory leak in Di...

2015-02-19 Thread nishkamravi2
Github user nishkamravi2 commented on the pull request: https://github.com/apache/spark/pull/4690#issuecomment-75018795 I have that commit. The problem continues to manifest itself. This is a workaround, not a fix at the source. I don't think the fix is trivial. --- If your project

[GitHub] spark pull request: [SPARK-5559] [Streaming] [Test] Remove oppotun...

2015-02-19 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/4337#issuecomment-75127417 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-5775] BugFix: GenericRow cannot be cast...

2015-02-19 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/4697#issuecomment-75128095 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-5559] [Streaming] [Test] Remove oppotun...

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

[GitHub] spark pull request: [SPARK-5775] BugFix: GenericRow cannot be cast...

2015-02-19 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/4697#issuecomment-75128074 /cc @liancheng --- 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-5775] BugFix: GenericRow cannot be cast...

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

[GitHub] spark pull request: [SPARK-5825] [Spark Submit] Remove the double ...

2015-02-19 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/4611#issuecomment-75128516 @andrewor14 Ah! right of course. I looked right past that. Yes that's a good change then. --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark pull request: [SPARK-1537][WIP]: ats support for spark-on-ya...

2015-02-19 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/4683#issuecomment-75128247 @zhzhan the problem is not creating 'alpha' features, but consuming them from elsewhere. These APIs don't exist at all until recent versions of YARN, so Spark needs

[GitHub] spark pull request: SPARK-4588 [MLLIB] [WIP] Add API for feature a...

2015-02-19 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/4460#issuecomment-75129379 I'm OK with a type hierarchy as long as it stays simple (and doesn't turn into a type system parallel to the DataFrame system). To support any type of

[GitHub] spark pull request: [SPARK-5904][SQL] DataFrame API fixes.

2015-02-19 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/4686#issuecomment-75129950 Btw, this fixed my issue with Java examples using DataFrame --- 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-5494][SQL] SparkSqlSerializer Ignores K...

2015-02-19 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/4693#issuecomment-75130162 There is no reason to allow custom kryo registration. The SQL serializer is only ever used to serialize SQL types. Have you seen some bug with us missing a type?

[GitHub] spark pull request: [SPARK-2555] Support configuration spark.sched...

2015-02-19 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/1462#discussion_r25022847 --- Diff: docs/configuration.md --- @@ -831,7 +831,7 @@ Apart from these, the following properties are also available, and may be useful td0/td

[GitHub] spark pull request: [SPARK-2555] Support configuration spark.sched...

2015-02-19 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/1462#issuecomment-75130879 @pwendell @kayousterhout what is the verdict of this? Should we just remove the ratio altogether? What about backward compatibility? --- If your project is set up

[GitHub] spark pull request: [SPARK-5902] [ml] Made PipelineStage.transform...

2015-02-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4682#issuecomment-75122689 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5900][MLLIB] make PIC and FPGrowth Java...

2015-02-19 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/4695#issuecomment-75123777 If we return a `JavaPairRDD`, the user code looks like the following: ~~~ for (Tuple2Long, Int assignment: assignments.collect()) { ... assignment._1()

[GitHub] spark pull request: [SPARK-5904][SQL] DataFrame API fixes.

2015-02-19 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/4686#issuecomment-75124929 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 not have this feature enabled

[GitHub] spark pull request: [SPARK-5825] [Spark Submit] Remove the double ...

2015-02-19 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/4611#issuecomment-75125804 @srowen it actually does a `=~` which should be equivalent to a grep: ``` if [[ abc =~ abc ]]; then echo woohoo; fi # woohoo if [[ ffabc =~ abc ]]; then

[GitHub] spark pull request: [SPARK-5873][SQL] Allow viewing of partially a...

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

[GitHub] spark pull request: [SPARK-4423] Improve foreach() documentation t...

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

[GitHub] spark pull request: [SPARK-4423] Improve foreach() documentation t...

2015-02-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4696#issuecomment-75119787 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5900][MLLIB] make PIC and FPGrowth Java...

2015-02-19 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/4695#issuecomment-75121366 For PIC, the names Assignment / id / cluster sound good to me. Those would be applicable to other clustering methods if ever needed. Assignment is a little generic,

[GitHub] spark pull request: [SPARK-1537][WIP]: ats support for spark-on-ya...

2015-02-19 Thread zhzhan
Github user zhzhan commented on the pull request: https://github.com/apache/spark/pull/4683#issuecomment-75124964 @srowen Yarn timeline is in alpha phase, but hadoop is embracing it as it provides a lot of extra functionalities. I will upload the patch and design doc to the JIRA.

[GitHub] spark pull request: [SPARK-5342][YARN] Allow long running Spark ap...

2015-02-19 Thread harishreedharan
Github user harishreedharan commented on the pull request: https://github.com/apache/spark/pull/4688#issuecomment-75124862 Jenkins, retest 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-5825] [Spark Submit] Remove the double ...

2015-02-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/4611 --- 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-5900][MLLIB] make PIC and FPGrowth Java...

2015-02-19 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/4695#issuecomment-75125808 LGTM except for that one comment --- 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-5904][SQL] DataFrame API fixes.

2015-02-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/4686 --- 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-5867] [SPARK-5892] [doc] [ml] [mllib] D...

2015-02-19 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/4675#issuecomment-75122512 @mbofb Thanks for noticing the issues with rows and cols documentation! I'll follow that JIRA. --- If your project is set up for it, you can reply to this email and

[GitHub] spark pull request: [SPARK-5902] [ml] Made PipelineStage.transform...

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

[GitHub] spark pull request: [SPARK-5342][YARN] Allow long running Spark ap...

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

[GitHub] spark pull request: [SPARK-5342][YARN] Allow long running Spark ap...

2015-02-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4688#issuecomment-75123043 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5900][MLLIB] make PIC and FPGrowth Java...

2015-02-19 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/4695#discussion_r25020128 --- Diff: docs/mllib-frequent-pattern-mining.md --- @@ -74,11 +74,12 @@ Calling `FPGrowth.run` with transactions returns an that stores the frequent

[GitHub] spark pull request: [SPARK-5900][MLLIB] make PIC and FPGrowth Java...

2015-02-19 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/4695#issuecomment-75125659 Yeah, after reading the code, I like the special classes since the field names make the code more legible. --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: [SPARK-5900][MLLIB] make PIC and FPGrowth Java...

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

[GitHub] spark pull request: [SPARK-5873][SQL] Allow viewing of partially a...

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

[GitHub] spark pull request: [SPARK-5347][CORE] Change FileSplit to InputSp...

2015-02-19 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/4150#issuecomment-75131653 Hi @shenh062326 since this is a duplicate would you mind closing this PR? The associated JIRA is already closed. Thanks. --- If your project is set up for it, you

[GitHub] spark pull request: [SPARK-2555] Support configuration spark.sched...

2015-02-19 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/1462#discussion_r25022830 --- Diff: core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerBackend.scala --- @@ -62,6 +62,11 @@ private[spark] class

[GitHub] spark pull request: [SPARK-5095] Support capping cores and launch ...

2015-02-19 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/4027#issuecomment-75131737 Hey @tnachen can you add `[Mesos]` to the title? --- 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-5873][SQL] Allow viewing of partially a...

2015-02-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4684#issuecomment-75131793 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: SPARK-5570: No docs stating that `new SparkCon...

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

[GitHub] spark pull request: [SPARK-5873][SQL] Allow viewing of partially a...

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

[GitHub] spark pull request: [SPARK-5902] [ml] Made PipelineStage.transform...

2015-02-19 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/4682#issuecomment-75132587 Merged into master and branch-1.3. 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

[GitHub] spark pull request: [SPARK-4808] Remove Spillable minimum threshol...

2015-02-19 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/3656#issuecomment-75132672 Hey @lawlerd this seems to be superseded by #4420. Would you mind closing this PR? Please follow the latest discussion there. --- If your project is set up for it,

[GitHub] spark pull request: [SPARK-5902] [ml] Made PipelineStage.transform...

2015-02-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/4682 --- 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-5570: No docs stating that `new SparkCon...

2015-02-19 Thread ilganeli
Github user ilganeli commented on the pull request: https://github.com/apache/spark/pull/4665#issuecomment-75132183 Hi @andrewor14 - Can you please review the language for the other driver options and let me know if you agree with it? Thanks! --- If your project is set up for it,

[GitHub] spark pull request: [SPARK-5900][MLLIB] make PIC and FPGrowth Java...

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

[GitHub] spark pull request: SPARK-4682 [CORE] Consolidate various 'Clock' ...

2015-02-19 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/4514#discussion_r25024948 --- Diff: core/src/main/scala/org/apache/spark/util/Clock.scala --- @@ -21,9 +21,36 @@ package org.apache.spark.util * An interface to represent clocks,

[GitHub] spark pull request: [SPARK-5900][MLLIB] make PIC and FPGrowth Java...

2015-02-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4695#issuecomment-75134621 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5873][SQL] Allow viewing of partially a...

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

[GitHub] spark pull request: [SPARK-5873][SQL] Allow viewing of partially a...

2015-02-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4684#issuecomment-75135416 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5846] Correctly set job description and...

2015-02-19 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/4631#issuecomment-75136332 Mind closing this now? --- 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-5846] Correctly set job description and...

2015-02-19 Thread kayousterhout
Github user kayousterhout closed the pull request at: https://github.com/apache/spark/pull/4631 --- 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

[GitHub] spark pull request: [SPARK-5881][SQL] RDD remains cached after the...

2015-02-19 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/4689#discussion_r24975506 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/CacheManager.scala --- @@ -39,7 +39,7 @@ private case class CachedData(plan: LogicalPlan,

[GitHub] spark pull request: [SPARK-5881][SQL] RDD remains cached after the...

2015-02-19 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/4689#issuecomment-75024215 This PR also added a `CLEAR CACHE` statement, would be good to add this in the PR description. --- If your project is set up for it, you can reply to this email and

[GitHub] spark pull request: [SPARK-5881][SQL] RDD remains cached after the...

2015-02-19 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/4689#discussion_r24977332 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/CachedTableSuite.scala --- @@ -280,4 +280,43 @@ class CachedTableSuite extends QueryTest {

[GitHub] spark pull request: Spark-5708: Add Slf4jSink to Spark Metrics

2015-02-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4644#issuecomment-75029758 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: Spark-5708: Add Slf4jSink to Spark Metrics

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

[GitHub] spark pull request: [SQL] Checking data types when resolving types

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

[GitHub] spark pull request: [SQL] Checking data types when resolving types

2015-02-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4685#issuecomment-75031054 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5882][GraphX] Add a test for GraphLoade...

2015-02-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4674#issuecomment-75034020 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5882][GraphX] Add a test for GraphLoade...

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

[GitHub] spark pull request: [SPARK-5881][SQL] RDD remains cached after the...

2015-02-19 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/4689#discussion_r24976568 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/SparkSQLParser.scala --- @@ -74,9 +75,15 @@ private[sql] class SparkSQLParser(fallback: String =

[GitHub] spark pull request: [SPARK-5881][SQL] RDD remains cached after the...

2015-02-19 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/4689#discussion_r24976046 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala --- @@ -888,6 +893,13 @@ class SQLContext(@transient val sparkContext:

[GitHub] spark pull request: [SPARK-5904][SQL] DataFrame API fixes.

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

[GitHub] spark pull request: [SPARK-5904][SQL] DataFrame API fixes.

2015-02-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4686#issuecomment-75026746 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5882][GraphX] Add a test for GraphLoade...

2015-02-19 Thread ankurdave
Github user ankurdave commented on the pull request: https://github.com/apache/spark/pull/4674#issuecomment-75022368 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-5436] [MLlib] Validate GradientBoostedT...

2015-02-19 Thread MechCoder
Github user MechCoder commented on the pull request: https://github.com/apache/spark/pull/4677#issuecomment-75026265 @jkbradley I have fixed up your comments. Btw, why are there are both a train and a run, which seems to me do the same thing. Is it not better to have one way

[GitHub] spark pull request: [SPARK-5882][GraphX] Add a test for GraphLoade...

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

[GitHub] spark pull request: SPARK-5841 [CORE] [HOTFIX 2] Memory leak in Di...

2015-02-19 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/4690#discussion_r24979807 --- Diff: core/src/main/scala/org/apache/spark/storage/DiskBlockManager.scala --- @@ -148,7 +148,13 @@ private[spark] class DiskBlockManager(blockManager:

[GitHub] spark pull request: [SPARK-5436] [MLlib] Validate GradientBoostedT...

2015-02-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4677#issuecomment-75029645 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5436] [MLlib] Validate GradientBoostedT...

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

[GitHub] spark pull request: [Spark-5889] Remove pid file after stopping se...

2015-02-19 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/4676#discussion_r24978988 --- Diff: sbin/spark-daemon.sh --- @@ -169,6 +169,7 @@ case $option in if [[ $(ps -p $TARGET_ID -o args=) =~ $command ]]; then echo

[GitHub] spark pull request: [SPARK-5436] [MLlib] Validate GradientBoostedT...

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

[GitHub] spark pull request: [SPARK-5436] [MLlib] Validate GradientBoostedT...

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

[GitHub] spark pull request: [SPARK-5436] [MLlib] Validate GradientBoostedT...

2015-02-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4677#issuecomment-75052509 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SQL] Checking data types when resolving types

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

[GitHub] spark pull request: [SPARK-5651][SQL] Add input64 in blacklist and...

2015-02-19 Thread watermen
GitHub user watermen reopened a pull request: https://github.com/apache/spark/pull/4427 [SPARK-5651][SQL] Add input64 in blacklist and add test suit for create table within backticks Now spark version is only support ```create table table_in_database_creation.test1 as select *

[GitHub] spark pull request: [SPARK-5651][SQL] Add input64 in blacklist and...

2015-02-19 Thread watermen
Github user watermen commented on the pull request: https://github.com/apache/spark/pull/4427#issuecomment-75016616 @marmbrus can you --- 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-5342][YARN] Allow long running Spark ap...

2015-02-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4688#issuecomment-75016589 **[Test build #27709 timed out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/27709/consoleFull)** for PR 4688 at commit

[GitHub] spark pull request: [SPARK-5342][YARN] Allow long running Spark ap...

2015-02-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4688#issuecomment-75016591 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5651][SQL] Add input64 in blacklist and...

2015-02-19 Thread watermen
Github user watermen closed the pull request at: https://github.com/apache/spark/pull/4427 --- 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-5494][SQL] SparkSqlSerializer Ignores K...

2015-02-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4693#issuecomment-75075748 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

  1   2   3   4   5   >