[GitHub] spark pull request: [SPARK-5775] [SQL] BugFix: GenericRow cannot b...

2015-02-27 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/4792#discussion_r25554933 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/parquet/newParquet.scala --- @@ -483,16 +487,42 @@ private[sql] case class ParquetRelation2(

[GitHub] spark pull request: [SPARK-6076][Block Manager] Fix a potential OO...

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

[GitHub] spark pull request: [SPARK-6077] update listener for the existing ...

2015-02-27 Thread zhichao-li
GitHub user zhichao-li opened a pull request: https://github.com/apache/spark/pull/4828 [SPARK-6077] update listener for the existing streamingTab instead of adding an one new one Currently we would create a new streaming tab for each streamingContext even if there's already one

[GitHub] spark pull request: [SPARK-4475] change localhost to 127.0.0.1...

2015-02-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/3425 --- 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: [CORE][YARN] SPARK-6011: Used Current Working ...

2015-02-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/4770 --- 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-3586][streaming]Support nested director...

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

[GitHub] spark pull request: [SPARK-3586][streaming]Support nested director...

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

[GitHub] spark pull request: [SPARK-6078][CORE] create event log dir if not...

2015-02-27 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/4829#issuecomment-76516159 This looks like a duplicate of #4120 / https://issues.apache.org/jira/browse/SPARK-5311, which we marked as Won't Fix:

[GitHub] spark pull request: [SPARK-5775] [SQL] BugFix: GenericRow cannot b...

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

[GitHub] spark pull request: [SPARK-5775] [SQL] BugFix: GenericRow cannot b...

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

[GitHub] spark pull request: [SPARK-6048] SparkConf should not translate de...

2015-02-27 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/4799#issuecomment-76505992 Given the discussion on the bug it looks ok, but it's missing the documentation that explains the behavior. --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: [SPARK-6074] [sql] Package pyspark sql binding...

2015-02-27 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/4822#issuecomment-76509136 That seems like the right way to do this thing. It mirrors similar declarations for core, mllib, streaming, etc. Should `tests.py` be in here? Maybe not worth bothering

[GitHub] spark pull request: [SPARK-6073][SQL] Need to refresh metastore ca...

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

[GitHub] spark pull request: [SPARK-6073][SQL] Need to refresh metastore ca...

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

[GitHub] spark pull request: [SPARK-5950][SQL]Insert array into a metastore...

2015-02-27 Thread yhuai
GitHub user yhuai opened a pull request: https://github.com/apache/spark/pull/4826 [SPARK-5950][SQL]Insert array into a metastore table saved as parquet should work when using datasource api This PR contains the following changes: 1. Add a new method,

[GitHub] spark pull request: [SPARK-4989][CORE] backport for branch-1.0(-jd...

2015-02-27 Thread liyezhang556520
Github user liyezhang556520 commented on the pull request: https://github.com/apache/spark/pull/3971#issuecomment-76515557 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

[GitHub] spark pull request: [SPARK-4989][CORE] backport for branch-1.0(-jd...

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

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

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

[GitHub] spark pull request: [SPARK-4226][SQL] SparkSQL - Add support for s...

2015-02-27 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/3249#discussion_r25552386 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -422,6 +424,108 @@ class Analyzer(catalog:

[GitHub] spark pull request: [SPARK-6066] Make event log format easier to p...

2015-02-27 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/4821#discussion_r25552048 --- Diff: core/src/main/scala/org/apache/spark/scheduler/EventLoggingListener.scala --- @@ -217,53 +219,60 @@ private[spark] object EventLoggingListener

[GitHub] spark pull request: [SPARK-6066] Make event log format easier to p...

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

[GitHub] spark pull request: [SPARK-6066] Make event log format easier to p...

2015-02-27 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/4821#discussion_r25552069 --- Diff: core/src/main/scala/org/apache/spark/scheduler/EventLoggingListener.scala --- @@ -202,11 +204,11 @@ private[spark] object EventLoggingListener

[GitHub] spark pull request: [CORE][minor] enhance the `toArray` method in ...

2015-02-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4825#issuecomment-76512123 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-6076][Block Manager] Fix a potential OO...

2015-02-27 Thread zsxwing
GitHub user zsxwing opened a pull request: https://github.com/apache/spark/pull/4827 [SPARK-6076][Block Manager] Fix a potential OOM issue when StorageLevel is MEMORY_AND_DISK_SER In

[GitHub] spark pull request: [SPARK-3586][streaming]Support nested director...

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

[GitHub] spark pull request: [SPARK-6078][CORE] create event log dir if not...

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

[GitHub] spark pull request: [SPARK-3586][streaming]Support nested director...

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

[GitHub] spark pull request: [SPARK-6050] [yarn] Add config option to do la...

2015-02-27 Thread mridulm
Github user mridulm commented on the pull request: https://github.com/apache/spark/pull/4818#issuecomment-76515153 @sryza When cpu scheduling is enabled (ref @tgravescs comment here and in jira) it must be validated. Just as we validate memory and while prioritizing based on

[GitHub] spark pull request: [SPARK-5950][SQL] Enable inserting array into ...

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

[GitHub] spark pull request: [SQL][WIP] Visualizing query plans with graphv...

2015-02-27 Thread pierre-borckmans
Github user pierre-borckmans commented on the pull request: https://github.com/apache/spark/pull/2354#issuecomment-76424493 Why was this deleted? --- 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-5522] Accelerate the Histroty Server st...

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

[GitHub] spark pull request: [SPARK-6055] [PySpark] fix incorrect __eq__ of...

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

[GitHub] spark pull request: [SPARK-6055] [PySpark] fix incorrect DataType....

2015-02-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4810#issuecomment-76425916 [QA tests have started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28073/consoleFull) for PR 4810 at commit

[GitHub] spark pull request: [SPARK-6029] Stop excluding fastutil package

2015-02-27 Thread jkleckner
Github user jkleckner commented on the pull request: https://github.com/apache/spark/pull/4780#issuecomment-76428126 Are you packaging CA yourself, are you marking all the Spark deps as provided? I see you're already using the ```userClassPathFirst``` settings. Cases of

[GitHub] spark pull request: [SPARK-5297][Streaming][backport] Backport SPA...

2015-02-27 Thread jerryshao
Github user jerryshao closed the pull request at: https://github.com/apache/spark/pull/4154 --- 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-4894][mllib] Added Bernoulli option to ...

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

[GitHub] spark pull request: [SPARK-5522] Accelerate the Histroty Server st...

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

[GitHub] spark pull request: [SPARK-5938][SQL] Generate Row from JSON strin...

2015-02-27 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/4712#issuecomment-76422284 @marmbrus do you have time to take a look of 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-5979][SPARK-6032] Smaller safer fix

2015-02-27 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/4802#issuecomment-76427925 Mind maybe copying some of the description from the other PR into this one so that the commit message is useful? --- If your project is set up for it, you can reply

[GitHub] spark pull request: [SPARK-5938][SQL] Generate Row from JSON strin...

2015-02-27 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/4712#issuecomment-76423117 Not until after the 1.3 release. On Feb 27, 2015 8:24 AM, Liang-Chi Hsieh notificati...@github.com wrote: @marmbrus https://github.com/marmbrus do you have

[GitHub] spark pull request: [SPARK-5297][Streaming][backport] Backport SPA...

2015-02-27 Thread jerryshao
Github user jerryshao commented on the pull request: https://github.com/apache/spark/pull/4154#issuecomment-76423609 OK, I will close this PR, thanks a lot for your comments. --- 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-6055] [PySpark] fix incorrect DataType....

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

[GitHub] spark pull request: SPARK-5983 [WEBUI] Don't respond to HTTP TRACE...

2015-02-27 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/4765#issuecomment-76393975 Going once going twice... I'd love to hear a slightly tidier way to handle this across the board, although, the code is just 3 lines duplicated in 3 clear places, so not

[GitHub] spark pull request: [SPARK-3586][streaming]Support nested director...

2015-02-27 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/2765#discussion_r25502732 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/api/java/JavaStreamingContext.scala --- @@ -279,13 +289,14 @@ class JavaStreamingContext(val

[GitHub] spark pull request: [SPARK-3586][streaming]Support nested director...

2015-02-27 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/2765#discussion_r25502749 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/api/java/JavaStreamingContext.scala --- @@ -309,12 +321,13 @@ class JavaStreamingContext(val

[GitHub] spark pull request: SPARK-1965 [WEBUI] Spark UI throws NPE on tryi...

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

[GitHub] spark pull request: [SPARK-3586][streaming]Support nested director...

2015-02-27 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/2765#discussion_r25502765 --- Diff: project/MimaExcludes.scala --- @@ -83,7 +83,16 @@ object MimaExcludes { // SPARK-2757

[GitHub] spark pull request: [SPARK-3586][streaming]Support nested director...

2015-02-27 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/2765#discussion_r25502811 --- Diff: python/pyspark/streaming/context.py --- @@ -260,6 +260,12 @@ def textFileStream(self, directory): return

[GitHub] spark pull request: SPARK-1965 [WEBUI] Spark UI throws NPE on tryi...

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

[GitHub] spark pull request: SPARK-1965 [WEBUI] Spark UI throws NPE on tryi...

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

[GitHub] spark pull request: [SPARK-5522] Accelerate the Histroty Server st...

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

[GitHub] spark pull request: [SPARK-5434] [EC2] Preserve spaces in EC2 path

2015-02-27 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/4224#issuecomment-76396713 I backported this to 1.2 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request: [SPARK-5522] Accelerate the Histroty Server st...

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

[GitHub] spark pull request: [SPARK-5522] Accelerate the Histroty Server st...

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

[GitHub] spark pull request: [SPARK-3586][streaming]Support nested director...

2015-02-27 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/2765#discussion_r25502824 --- Diff: python/pyspark/streaming/tests.py --- @@ -446,7 +446,7 @@ def test_queue_stream(self): def test_text_file_stream(self): d =

[GitHub] spark pull request: SPARK-1965 [WEBUI] Spark UI throws NPE on tryi...

2015-02-27 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/4777#issuecomment-76387038 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: [SPARK-6059][Yarn] Add volatile to Application...

2015-02-27 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/4814#issuecomment-76388516 I'm going to merge into master only just because it's not clear there is an actual practical issue as a result of this. --- If your project is set up for it, you can

[GitHub] spark pull request: [SPARK-6058][Yarn] Log the user class exceptio...

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

[GitHub] spark pull request: [SPARK-6058][Yarn] Log the user class exceptio...

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

[GitHub] spark pull request: [SPARK-6058][Yarn] Log the user class exceptio...

2015-02-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/4813 --- 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-1965 [WEBUI] Spark UI throws NPE on tryi...

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

[GitHub] spark pull request: SPARK-1965 [WEBUI] Spark UI throws NPE on tryi...

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

[GitHub] spark pull request: [SPARK-6059][Yarn] Add volatile to Application...

2015-02-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/4814 --- 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-5417] Remove redundant executor-id set(...

2015-02-27 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/4213#issuecomment-76402017 Since the commit this depends on was in 1.2 (https://github.com/apache/spark/commit/1f8b7186813643afdb592131bfd89228ef971e74) and this was tagged for back-port into 1.2,

[GitHub] spark pull request: [SPARK-6055] [PySpark] fix incorrect DataType....

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

[GitHub] spark pull request: [SPARK-5847] Allow for namespacing metrics by ...

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

[GitHub] spark pull request: [SPARK-5751] [SQL] Sets SPARK_HOME as SPARK_PI...

2015-02-27 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/4758#discussion_r25526262 --- Diff: sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/HiveThriftServer2Suites.scala --- @@ -315,7 +315,14 @@ abstract

[GitHub] spark pull request: [SPARK-5751] [SQL] Sets SPARK_HOME as SPARK_PI...

2015-02-27 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/4758#issuecomment-76444830 Updated, you can merge it after Jenkins nods :) --- 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-6063

2015-02-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4815#issuecomment-76445667 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-6055] [PySpark] fix incorrect __eq__ of...

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

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

2015-02-27 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/4688#discussion_r25527424 --- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/ApplicationMaster.scala --- @@ -256,6 +256,12 @@ private[spark] class ApplicationMaster(

[GitHub] spark pull request: [SPARK-4587] [mllib] [docs] Fixed save,load ca...

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

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

2015-02-27 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/4688#discussion_r25525230 --- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala --- @@ -313,6 +320,21 @@ private[spark] class Client( env(SPARK_YARN_MODE)

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

2015-02-27 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/4688#discussion_r25526051 --- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/ClientArguments.scala --- @@ -222,6 +224,14 @@ private[spark] class ClientArguments(args:

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

2015-02-27 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/4688#discussion_r25526204 --- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/ApplicationMaster.scala --- @@ -256,6 +256,12 @@ private[spark] class ApplicationMaster(

[GitHub] spark pull request: [SPARK-6055] [PySpark] fix incorrect DataType....

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

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

2015-02-27 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/4688#discussion_r25526105 --- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/ApplicationMaster.scala --- @@ -256,6 +256,12 @@ private[spark] class ApplicationMaster(

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

2015-02-27 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/4688#discussion_r25526585 --- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala --- @@ -540,6 +561,29 @@ private[spark] class Client( amContainer }

[GitHub] spark pull request: [SPARK-6055] [PySpark] fix incorrect DataType....

2015-02-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4810#issuecomment-76441573 [QA tests have finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28073/consoleFull) for PR 4810 at commit

[GitHub] spark pull request: [SPARK-6055] [PySpark] fix incorrect DataType....

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

[GitHub] spark pull request: [SPARK-5847] Allow for namespacing metrics by ...

2015-02-27 Thread ryan-williams
Github user ryan-williams commented on the pull request: https://github.com/apache/spark/pull/4632#issuecomment-76444840 This one is ready for review, I think --- 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-6063

2015-02-27 Thread msjgriffiths
GitHub user msjgriffiths opened a pull request: https://github.com/apache/spark/pull/4815 SPARK-6063 Remove unicode characters from MLlib file. You can merge this pull request into a Git repository by running: $ git pull https://github.com/msjgriffiths/spark SPARK-6063

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

2015-02-27 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/4688#discussion_r25527152 --- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnSparkHadoopUtil.scala --- @@ -82,6 +93,102 @@ class YarnSparkHadoopUtil extends

[GitHub] spark pull request: [SPARK-4587] [mllib] [docs] Fixed save,load ca...

2015-02-27 Thread jkbradley
GitHub user jkbradley opened a pull request: https://github.com/apache/spark/pull/4816 [SPARK-4587] [mllib] [docs] Fixed save,load calls in ML guide examples Should pass spark context to save/load CC: @mengxr You can merge this pull request into a Git repository by

[GitHub] spark pull request: [SPARK-4587] [mllib] [docs] Fixed save,load ca...

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

[GitHub] spark pull request: [SPARK-6055] [PySpark] fix incorrect __eq__ of...

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

[GitHub] spark pull request: [SPARK-4894][mllib] Added Bernoulli option to ...

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

[GitHub] spark pull request: [SPARK-4894][mllib] Added Bernoulli option to ...

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

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

2015-02-27 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/4688#discussion_r25525993 --- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnSparkHadoopUtil.scala --- @@ -82,6 +93,102 @@ class YarnSparkHadoopUtil extends SparkHadoopUtil

[GitHub] spark pull request: [SPARK-5751] [SQL] Sets SPARK_HOME as SPARK_PI...

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

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

2015-02-27 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/4688#discussion_r25526812 --- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala --- @@ -866,23 +910,11 @@ object Client extends Logging { * Get the list of

[GitHub] spark pull request: [SPARK-6055] [PySpark] fix incorrect __eq__ of...

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

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

2015-02-27 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/4688#discussion_r25525865 --- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala --- @@ -313,6 +320,21 @@ private[spark] class Client( env(SPARK_YARN_MODE)

[GitHub] spark pull request: [SPARK-5991][MLLIB] support save/load in PySpa...

2015-02-27 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/4811#issuecomment-76446225 I messed up not passing sc to save/load. Is this patch going into 1.3? If not, then I'll submit a separate patch fixing the documentation (which will conflict a

[GitHub] spark pull request: SPARK-1965 [WEBUI] Spark UI throws NPE on tryi...

2015-02-27 Thread kayousterhout
Github user kayousterhout commented on the pull request: https://github.com/apache/spark/pull/4777#issuecomment-76448316 I don't think it's worth it to return a 404, and this is more consistent with how we treat stages/jobs that don't exist. I might say No running application with

[GitHub] spark pull request: SPARK-6063

2015-02-27 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/4815#issuecomment-76450569 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-5342][YARN] Allow long running Spark ap...

2015-02-27 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/4688#discussion_r25526548 --- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala --- @@ -540,6 +561,29 @@ private[spark] class Client( amContainer }

[GitHub] spark pull request: [SPARK-6051][Streaming] Add ZooKeeper offest p...

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

[GitHub] spark pull request: [SPARK-5522] Accelerate the Histroty Server st...

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

[GitHub] spark pull request: [SPARK-5950][SQL] Enable inserting array into ...

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

<    1   2   3   4   5   >