[GitHub] spark pull request: [SPARK-14868][BUILD] Enable NewLineAtEofChecke...

2016-04-23 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/12632#discussion_r60839977 --- Diff: mllib/src/test/java/org/apache/spark/ml/classification/JavaRandomForestClassifierSuite.java --- @@ -81,15 +81,15 @@ public void runDT() {

[GitHub] spark pull request: [SPARK-14867][BUILD] Remove `--force` option i...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12631#issuecomment-213905832 **[Test build #56836 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56836/consoleFull)** for PR 12631 at commit [`b11de8a`](https://gi

[GitHub] spark pull request: [SPARK-14865][SQL] Better error handling for v...

2016-04-23 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/12633#discussion_r60839961 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala --- @@ -85,68 +88,74 @@ case class CreateViewCommand( } else

[GitHub] spark pull request: [SPARK-14883][DOCS] Fix wrong R examples and m...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12649#issuecomment-213905769 **[Test build #56835 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56835/consoleFull)** for PR 12649 at commit [`5d6d45e`](https://gi

[GitHub] spark pull request: [SPARK-14876][SQL] SparkSession should be case...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12643#issuecomment-213905767 **[Test build #56834 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56834/consoleFull)** for PR 12643 at commit [`fd95cb7`](https://gi

[GitHub] spark pull request: [SPARK-14868][BUILD] Enable NewLineAtEofChecke...

2016-04-23 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/12632#discussion_r60839935 --- Diff: core/src/main/java/org/apache/spark/util/collection/unsafe/sort/PrefixComparators.java --- @@ -82,37 +81,37 @@ public static long computePre

[GitHub] spark pull request: [SPARK-14883][DOCS] Fix wrong R examples and m...

2016-04-23 Thread dongjoon-hyun
GitHub user dongjoon-hyun opened a pull request: https://github.com/apache/spark/pull/12649 [SPARK-14883][DOCS] Fix wrong R examples and make them up-to-date ## What changes were proposed in this pull request? This issue aims to fix some errors in R examples and make them up

[GitHub] spark pull request: [SPARK-14721][SQL] Remove HiveContext (part 2)

2016-04-23 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12585#issuecomment-213905492 The changes look good to me. --- 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-14868][BUILD] Enable NewLineAtEofChecke...

2016-04-23 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/12632#discussion_r60839755 --- Diff: core/src/main/java/org/apache/spark/util/collection/unsafe/sort/PrefixComparators.java --- @@ -82,37 +81,37 @@ public static long computePrefix(doubl

[GitHub] spark pull request: [SPARK-14868][BUILD] Enable NewLineAtEofChecke...

2016-04-23 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/12632#discussion_r60839760 --- Diff: mllib/src/test/java/org/apache/spark/ml/classification/JavaRandomForestClassifierSuite.java --- @@ -81,15 +81,15 @@ public void runDT() { fo

[GitHub] spark pull request: [SPARK-14487][SQL] User Defined Type registrat...

2016-04-23 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/12259#issuecomment-213897223 @mengxr @dbtsai Your comments are addressed now. Please take a look again when you have a change. Thanks. --- If your project is set up for it, you can reply to this em

[GitHub] spark pull request: [SPARK-14876][SQL] SparkSession should be case...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12643#issuecomment-213896260 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 projec

[GitHub] spark pull request: [SPARK-14876][SQL] SparkSession should be case...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12643#issuecomment-213896261 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14876][SQL] SparkSession should be case...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12643#issuecomment-213896243 **[Test build #56833 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56833/consoleFull)** for PR 12643 at commit [`40aefc0`](https://g

[GitHub] spark pull request: [SPARK-14867][BUILD] Remove `--force` option i...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12631#issuecomment-213892900 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 projec

[GitHub] spark pull request: [SPARK-14867][BUILD] Remove `--force` option i...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12631#issuecomment-213892901 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14867][BUILD] Remove `--force` option i...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12631#issuecomment-213892810 **[Test build #56830 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56830/consoleFull)** for PR 12631 at commit [`8504cce`](https://g

[GitHub] spark pull request: [SPARK-14548][SQL] Support not greater than an...

2016-04-23 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12316#issuecomment-213892774 @hvanhovell Can you think of any downsides if we support this? It is a little bit esoteric. The main one I can think of is whether this would set a precedent in that a lot

[GitHub] spark pull request: [SPARK-14881][PYTHON][SPARKR] pyspark and spar...

2016-04-23 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/spark/pull/12648#issuecomment-213892725 Hmmm... ``` [info] Run completed in 1 hour, 18 minutes, 30 seconds. [info] Total number of tests run: 1805 [info] Suites: completed 65, aborted 0

[GitHub] spark pull request: Support single argument version of sqlContext....

2016-04-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12488 --- 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 ena

[GitHub] spark pull request: [SPARK-14867][BUILD] Remove `--force` option i...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12631#issuecomment-213892530 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14867][BUILD] Remove `--force` option i...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12631#issuecomment-213892529 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 projec

[GitHub] spark pull request: Support single argument version of sqlContext....

2016-04-23 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12488#issuecomment-213892519 hmm is there a JIRA ticket associated with this that you have created? --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[GitHub] spark pull request: Support single argument version of sqlContext....

2016-04-23 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12488#issuecomment-213892503 Thanks - merging 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 have

[GitHub] spark pull request: [SPARK-14867][BUILD] Remove `--force` option i...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12631#issuecomment-213892482 **[Test build #56829 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56829/consoleFull)** for PR 12631 at commit [`cfe9b4e`](https://g

[GitHub] spark pull request: [SPARK-14487][SQL] User Defined Type registrat...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12259#issuecomment-213892427 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 projec

[GitHub] spark pull request: [SPARK-14487][SQL] User Defined Type registrat...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12259#issuecomment-213892428 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14487][SQL] User Defined Type registrat...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12259#issuecomment-213892392 **[Test build #56831 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56831/consoleFull)** for PR 12259 at commit [`06bdbc5`](https://g

[GitHub] spark pull request: [SPARK-14879] [SQL] Move CreateMetastoreDataSo...

2016-04-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12645 --- 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 ena

[GitHub] spark pull request: [SPARK-14879] [SQL] Move CreateMetastoreDataSo...

2016-04-23 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/12645#issuecomment-213892106 OK. Thanks. Will send out a follow-up pr. --- 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-14879] [SQL] Move CreateMetastoreDataSo...

2016-04-23 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/12645#discussion_r60839199 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -0,0 +1,452 @@ +/* + * Licensed to the Ap

[GitHub] spark pull request: [SPARK-14879] [SQL] Move CreateMetastoreDataSo...

2016-04-23 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12645#issuecomment-213891570 Most of the problems I pointed out also existed in the old code, so feel free to merge this one and submit a follow-up pr to address them. --- If your project is set up

[GitHub] spark pull request: [SPARK-14879] [SQL] Move CreateMetastoreDataSo...

2016-04-23 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/12645#discussion_r60839135 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -0,0 +1,452 @@ +/* + * Licensed to the Apa

[GitHub] spark pull request: [SPARK-14879] [SQL] Move CreateMetastoreDataSo...

2016-04-23 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/12645#discussion_r60839130 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -0,0 +1,452 @@ +/* + * Licensed to the Apa

[GitHub] spark pull request: [SPARK-14879] [SQL] Move CreateMetastoreDataSo...

2016-04-23 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/12645#discussion_r60839110 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -0,0 +1,452 @@ +/* + * Licensed to the Apa

[GitHub] spark pull request: [SPARK-14879] [SQL] Move CreateMetastoreDataSo...

2016-04-23 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/12645#discussion_r60839082 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -0,0 +1,452 @@ +/* + * Licensed to the Apa

[GitHub] spark pull request: [SPARK-13902][SPARK-14269][SCHEDULER] Eliminat...

2016-04-23 Thread kayousterhout
Github user kayousterhout commented on the pull request: https://github.com/apache/spark/pull/12060#issuecomment-213890432 In theory this seems like a good idea: there do seem to be lots of unnecessary calls to submitWaitingStages(), and those calls seem somewhat expensive since they

[GitHub] spark pull request: [SPARK-14879] [SQL] Move CreateMetastoreDataSo...

2016-04-23 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/12645#discussion_r60839067 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -0,0 +1,452 @@ +/* + * Licensed to the Apa

[GitHub] spark pull request: [SPARK-14879] [SQL] Move CreateMetastoreDataSo...

2016-04-23 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/12645#discussion_r60838889 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -0,0 +1,452 @@ +/* + * Licensed to the Apa

[GitHub] spark pull request: [SPARK-14879] [SQL] Move CreateMetastoreDataSo...

2016-04-23 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/12645#discussion_r60838884 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -0,0 +1,452 @@ +/* + * Licensed to the Apa

[GitHub] spark pull request: [SPARK-14833][SQL][STREAMING][TEST] Refactor S...

2016-04-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12592 --- 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 ena

[GitHub] spark pull request: [SPARK-14879] [SQL] Move CreateMetastoreDataSo...

2016-04-23 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/12645#discussion_r60838834 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -0,0 +1,452 @@ +/* + * Licensed to the Apa

[GitHub] spark pull request: [SPARK-14833][SQL][STREAMING][TEST] Refactor S...

2016-04-23 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/12592#issuecomment-213889929 LGTM. Thanks, merging to 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 no

[GitHub] spark pull request: [SPARK-13902][SPARK-14269][SCHEDULER] Eliminat...

2016-04-23 Thread kayousterhout
Github user kayousterhout commented on a diff in the pull request: https://github.com/apache/spark/pull/12060#discussion_r60838822 --- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala --- @@ -403,32 +403,47 @@ class DAGScheduler( parents.toList

[GitHub] spark pull request: [SPARK-14879] [SQL] Move CreateMetastoreDataSo...

2016-04-23 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/12645#discussion_r60838810 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -0,0 +1,452 @@ +/* + * Licensed to the Apa

[GitHub] spark pull request: [SPARK-14879] [SQL] Move CreateMetastoreDataSo...

2016-04-23 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/12645#discussion_r60838743 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -0,0 +1,452 @@ +/* + * Licensed to the Apa

[GitHub] spark pull request: [SPARK-14879] [SQL] Move CreateMetastoreDataSo...

2016-04-23 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/12645#discussion_r60838739 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -0,0 +1,452 @@ +/* + * Licensed to the Apa

[GitHub] spark pull request: [SPARK-14876][SQL] SparkSession should be case...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12643#issuecomment-213888417 **[Test build #56833 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56833/consoleFull)** for PR 12643 at commit [`40aefc0`](https://gi

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2016-04-23 Thread yhuai
Github user yhuai closed the pull request at: https://github.com/apache/spark/pull/9702 --- 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 enabl

[GitHub] spark pull request: [WIP] [SQL] Towards removing HiveContext

2016-04-23 Thread yhuai
Github user yhuai closed the pull request at: https://github.com/apache/spark/pull/12410 --- 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 enab

[GitHub] spark pull request: [SQL] How many tests will fail if we do not ha...

2016-04-23 Thread yhuai
Github user yhuai closed the pull request at: https://github.com/apache/spark/pull/12367 --- 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 enab

[GitHub] spark pull request: [SQL] Can we remove executionHive?

2016-04-23 Thread yhuai
Github user yhuai closed the pull request at: https://github.com/apache/spark/pull/12372 --- 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 enab

[GitHub] spark pull request: [SPARK-14505] [Core] Fix bug : creating two Sp...

2016-04-23 Thread the-sea
Github user the-sea commented on the pull request: https://github.com/apache/spark/pull/12273#issuecomment-213882250 @srowen sorry for the delay, I have free time only on weekends -_-! --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[GitHub] spark pull request: [SPARK-14838][SQL] Set default size for ObjecT...

2016-04-23 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/12599#issuecomment-213881860 Unrelated failure. I think it is ok. 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 proj

[GitHub] spark pull request: [SPARK-14838][SQL] Set default size for ObjecT...

2016-04-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12599 --- 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 ena

[GitHub] spark pull request: [SPARK-14838][SQL] Set default size for ObjecT...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12599#issuecomment-213881827 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14838][SQL] Set default size for ObjecT...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12599#issuecomment-213881826 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 projec

[GitHub] spark pull request: [SPARK-14838][SQL] Set default size for ObjecT...

2016-04-23 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/12599#issuecomment-213881630 LGTM, Merging this into master, 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 proj

[GitHub] spark pull request: [SPARK-14487][SQL] User Defined Type registrat...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12259#issuecomment-213881371 **[Test build #56831 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56831/consoleFull)** for PR 12259 at commit [`06bdbc5`](https://gi

[GitHub] spark pull request: [SPARK-14098][SQL] Generate Java code that get...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213881135 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 projec

[GitHub] spark pull request: [SPARK-14098][SQL] Generate Java code that get...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213881140 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14098][SQL] Generate Java code that get...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213881027 **[Test build #56827 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56827/consoleFull)** for PR 11956 at commit [`cd8855f`](https://g

[GitHub] spark pull request: [SPARK-14487][SQL] User Defined Type registrat...

2016-04-23 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/12259#issuecomment-213881020 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 fe

[GitHub] spark pull request: [SPARK-14487][SQL] User Defined Type registrat...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12259#issuecomment-213880954 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14838][SQL] Set default size for ObjecT...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12599#issuecomment-213880968 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14838][SQL] Set default size for ObjecT...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12599#issuecomment-213880967 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 projec

[GitHub] spark pull request: [SPARK-14487][SQL] User Defined Type registrat...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12259#issuecomment-213880953 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 projec

[GitHub] spark pull request: [SPARK-14838][SQL] Set default size for ObjecT...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12599#issuecomment-213880939 **[Test build #56825 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56825/consoleFull)** for PR 12599 at commit [`c26c3bd`](https://g

[GitHub] spark pull request: [SPARK-14867][BUILD] Remove `--force` option i...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12631#issuecomment-213880951 **[Test build #56830 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56830/consoleFull)** for PR 12631 at commit [`8504cce`](https://gi

[GitHub] spark pull request: [SPARK-14487][SQL] User Defined Type registrat...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12259#issuecomment-213880936 **[Test build #56828 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56828/consoleFull)** for PR 12259 at commit [`06bdbc5`](https://g

[GitHub] spark pull request: [SPARK-14867][BUILD] Remove `--force` option i...

2016-04-23 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/12631#issuecomment-213880918 Rebased. --- 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 featur

[GitHub] spark pull request: [SPARK-14874][SQL][Streaming] Remove the obsol...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12638#issuecomment-213879726 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 projec

[GitHub] spark pull request: [SPARK-14874][SQL][Streaming] Remove the obsol...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12638#issuecomment-213879728 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14874][SQL][Streaming] Remove the obsol...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12638#issuecomment-213879699 **[Test build #56824 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56824/consoleFull)** for PR 12638 at commit [`c79cba9`](https://g

[GitHub] spark pull request: [SPARK-14867][BUILD] Remove `--force` option i...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12631#issuecomment-213877402 **[Test build #56829 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56829/consoleFull)** for PR 12631 at commit [`cfe9b4e`](https://gi

[GitHub] spark pull request: [SPARK-14487][SQL] User Defined Type registrat...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12259#issuecomment-213873676 **[Test build #56828 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56828/consoleFull)** for PR 12259 at commit [`06bdbc5`](https://gi

[GitHub] spark pull request: [SPARK-14487][SQL] User Defined Type registrat...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12259#issuecomment-213872213 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 projec

[GitHub] spark pull request: [SPARK-14487][SQL] User Defined Type registrat...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12259#issuecomment-213872211 **[Test build #56826 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56826/consoleFull)** for PR 12259 at commit [`3917f6b`](https://g

[GitHub] spark pull request: [SPARK-14487][SQL] User Defined Type registrat...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12259#issuecomment-213872215 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14487][SQL] User Defined Type registrat...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12259#issuecomment-213872166 **[Test build #56826 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56826/consoleFull)** for PR 12259 at commit [`3917f6b`](https://gi

[GitHub] spark pull request: [SPARK-14098][SQL] Generate Java code that get...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-213872164 **[Test build #56827 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56827/consoleFull)** for PR 11956 at commit [`cd8855f`](https://gi

[GitHub] spark pull request: [SPARK-14838][SQL] Set default size for ObjecT...

2016-04-23 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/12599#discussion_r60838031 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/DatasetSuite.scala --- @@ -630,6 +630,29 @@ class DatasetSuite extends QueryTest with SharedSQLContext

[GitHub] spark pull request: [SPARK-14838][SQL] Set default size for ObjecT...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12599#issuecomment-213870976 **[Test build #56825 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56825/consoleFull)** for PR 12599 at commit [`c26c3bd`](https://gi

[GitHub] spark pull request: [SPARK-14874][SQL][Streaming] Remove the obsol...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12638#issuecomment-213870648 **[Test build #56824 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56824/consoleFull)** for PR 12638 at commit [`c79cba9`](https://gi

[GitHub] spark pull request: [SPARK-14874][SQL][Streaming] Remove the obsol...

2016-04-23 Thread lw-lin
Github user lw-lin commented on the pull request: https://github.com/apache/spark/pull/12638#issuecomment-213870571 @marmbrus @tdas would you mind taking a look? Thanks! :-) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] spark pull request: [SPARK-14487][SQL] User Defined Type registrat...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12259#issuecomment-213870537 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 projec

[GitHub] spark pull request: [SPARK-14487][SQL] User Defined Type registrat...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12259#issuecomment-213870536 **[Test build #56823 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56823/consoleFull)** for PR 12259 at commit [`023d281`](https://g

[GitHub] spark pull request: [SPARK-14487][SQL] User Defined Type registrat...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12259#issuecomment-213870539 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14874][SQL][Streaming] Remove the obsol...

2016-04-23 Thread lw-lin
GitHub user lw-lin reopened a pull request: https://github.com/apache/spark/pull/12638 [SPARK-14874][SQL][Streaming] Remove the obsolete Batch representation ## What changes were proposed in this pull request? The `Batch` class, which had been used to indicate progress in a

[GitHub] spark pull request: [SPARK-14487][SQL] User Defined Type registrat...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12259#issuecomment-213870512 **[Test build #56823 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56823/consoleFull)** for PR 12259 at commit [`023d281`](https://gi

[GitHub] spark pull request: [SPARK-14487][SQL] User Defined Type registrat...

2016-04-23 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/12259#discussion_r60837881 --- Diff: mllib/src/test/scala/org/apache/spark/ml/linalg/MatrixUDTSuite.scala --- @@ -0,0 +1,68 @@ +/* + * Licensed to the Apache Software Foundation

[GitHub] spark pull request: [SPARK-14881][PYTHON][SPARKR] pyspark and spar...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12648#issuecomment-213864239 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 projec

[GitHub] spark pull request: [SPARK-14881][PYTHON][SPARKR] pyspark and spar...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12648#issuecomment-213864240 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14881][PYTHON][SPARKR] pyspark and spar...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12648#issuecomment-213864100 **[Test build #56822 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56822/consoleFull)** for PR 12648 at commit [`005892a`](https://g

[GitHub] spark pull request: [SPARK-12148][SPARKR] fix doc after renaming D...

2016-04-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12647 --- 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 ena

[GitHub] spark pull request: [SPARK-12148][SPARKR] fix doc after renaming D...

2016-04-23 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12647#issuecomment-213862448 Thanks - merging 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 have

[GitHub] spark pull request: [SPARK-14838][SQL] Set default size for ObjecT...

2016-04-23 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/12599#discussion_r60837461 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/DatasetSuite.scala --- @@ -630,6 +630,29 @@ class DatasetSuite extends QueryTest with SharedSQLContext {

[GitHub] spark pull request: [SPARK-14838][SQL] Set default size for ObjecT...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12599#issuecomment-213862390 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 projec

[GitHub] spark pull request: [SPARK-14838][SQL] Set default size for ObjecT...

2016-04-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12599#issuecomment-213862394 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-14838][SQL] Set default size for ObjecT...

2016-04-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12599#issuecomment-213862360 **[Test build #56820 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56820/consoleFull)** for PR 12599 at commit [`6b6c12d`](https://g

  1   2   3   4   >