[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221489454 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-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221489457 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-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221489377 **[Test build #59263 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59263/consoleFull)** for PR 13284 at commit [`3028e2c`](https://g

[GitHub] spark pull request: [SPARK-15100][DOC] Modified user guide and exa...

2016-05-24 Thread MLnick
Github user MLnick commented on a diff in the pull request: https://github.com/apache/spark/pull/13176#discussion_r64523585 --- Diff: docs/ml-features.md --- @@ -151,7 +151,7 @@ for more details on the API. term frequency across the corpus. An optional parameter `minDF` also

[GitHub] spark pull request: [SPARK-15493][SQL] Allow setting the quoteEsca...

2016-05-24 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/13267#issuecomment-221489074 BTW don'r forget to update the title too. --- 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-15100][DOC] Modified user guide and exa...

2016-05-24 Thread MLnick
Github user MLnick commented on a diff in the pull request: https://github.com/apache/spark/pull/13176#discussion_r64523407 --- Diff: docs/ml-features.md --- @@ -1100,7 +1100,7 @@ for more details on the API. categorical features. The number of bins is set by the `numBuckets`

[GitHub] spark pull request: [SPARK-15100][DOC] Modified user guide and exa...

2016-05-24 Thread MLnick
Github user MLnick commented on a diff in the pull request: https://github.com/apache/spark/pull/13176#discussion_r64523300 --- Diff: docs/ml-features.md --- @@ -1098,9 +1098,9 @@ for more details on the API. `QuantileDiscretizer` takes a column with continuous features a

[GitHub] spark pull request: [SPARK-13484][SQL] Prevent illegal NULL propag...

2016-05-24 Thread maropu
Github user maropu commented on a diff in the pull request: https://github.com/apache/spark/pull/11371#discussion_r64522917 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -1443,6 +1445,32 @@ class Analyzer( }

[GitHub] spark pull request: [SPARK-15493][SQL] Allow setting the quoteEsca...

2016-05-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13267#issuecomment-221487940 **[Test build #3018 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3018/consoleFull)** for PR 13267 at commit [`8c4bef1`](https://g

[GitHub] spark pull request: [SPARK-15092][SPARK-15139][PYSPARK][ML] Pyspar...

2016-05-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12919#issuecomment-221487845 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-15092][SPARK-15139][PYSPARK][ML] Pyspar...

2016-05-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12919#issuecomment-221487773 **[Test build #59262 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59262/consoleFull)** for PR 12919 at commit [`6551fb4`](https://g

[GitHub] spark pull request: [SPARK-15092][SPARK-15139][PYSPARK][ML] Pyspar...

2016-05-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12919#issuecomment-221487843 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-15493][SQL] Allow setting the quoteEsca...

2016-05-24 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/13267#discussion_r64522811 --- Diff: python/pyspark/sql/readwriter.py --- @@ -787,6 +787,9 @@ def csv(self, path, mode=None, compression=None, sep=None, quote=None, escape=No

[GitHub] spark pull request: [SPARK-15493][SQL] Allow setting the quoteEsca...

2016-05-24 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/13267#discussion_r64522859 --- Diff: sql/core/pom.xml --- @@ -39,7 +39,7 @@ com.univocity univocity-parsers - 2.1.0 + 2.1.1 --- End di

[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread wangmiao1981
Github user wangmiao1981 commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221487340 @vectorijk please run conflicts(detail = TRUE) and check the $package:SparkR with test_context.R, namesOfMasked. 3.2.0 should have more methods than 3.1.0. -

[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221486881 **[Test build #59263 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59263/consoleFull)** for PR 13284 at commit [`3028e2c`](https://gi

[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread vectorijk
Github user vectorijk commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221486634 > @shivaram I will make the change with R version check. @wangmiao1981 FYI, I have switched to R 3.1.3 before on Mac. It seems fail too. Do you mind try it

[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221486545 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-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221486544 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-15493][SQL] Allow setting the quoteEsca...

2016-05-24 Thread jurriaan
Github user jurriaan commented on the pull request: https://github.com/apache/spark/pull/13267#issuecomment-221486596 @rxin Done :) --- 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-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221486456 **[Test build #59260 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59260/consoleFull)** for PR 13284 at commit [`4a187ca`](https://g

[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread wangmiao1981
Github user wangmiao1981 commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221486087 @vectorijk As we discussed above, R version is different on local and Jenkins. We installed R 3.3.0 in local while Jenkins still uses the old version. --- If your

[GitHub] spark pull request: [SPARK-15092][SPARK-15139][PYSPARK][ML] Pyspar...

2016-05-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12919#issuecomment-221486047 **[Test build #59262 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59262/consoleFull)** for PR 12919 at commit [`6551fb4`](https://gi

[GitHub] spark pull request: [SPARK-15092][SPARK-15139][PYSPARK][ML] Pyspar...

2016-05-24 Thread MLnick
Github user MLnick commented on the pull request: https://github.com/apache/spark/pull/12919#issuecomment-221485607 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-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread vectorijk
Github user vectorijk commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221485421 We may also investigate why unit test run differently on Jenkins and local. --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] spark pull request: [SQL] Prevent illegal NULL propagation when fi...

2016-05-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13290#issuecomment-221484525 **[Test build #59261 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59261/consoleFull)** for PR 13290 at commit [`fff3382`](https://gi

[GitHub] spark pull request: [SPARK-13484][SQL] Prevent illegal NULL propag...

2016-05-24 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/11371#discussion_r64521117 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -1443,6 +1445,32 @@ class Analyzer( } /

[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread wangmiao1981
Github user wangmiao1981 commented on a diff in the pull request: https://github.com/apache/spark/pull/13284#discussion_r64520919 --- Diff: R/pkg/inst/tests/testthat/test_context.R --- @@ -24,10 +24,18 @@ test_that("Check masked functions", { func <- lapply(masked, function(x

[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221483871 Ok - lets just fix the masking and the subset ones in this PR. Once we understand what is the problem with `pipeRDD` we can fix it in a separate PR. --- If your proje

[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread shivaram
Github user shivaram commented on a diff in the pull request: https://github.com/apache/spark/pull/13284#discussion_r64520757 --- Diff: R/pkg/inst/tests/testthat/test_context.R --- @@ -24,10 +24,18 @@ test_that("Check masked functions", { func <- lapply(masked, function(x) {

[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221483796 **[Test build #59260 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59260/consoleFull)** for PR 13284 at commit [`4a187ca`](https://gi

[GitHub] spark pull request: [SQL] Prevent illegal NULL propagation when fi...

2016-05-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13290#issuecomment-221483773 **[Test build #59259 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59259/consoleFull)** for PR 13290 at commit [`3cec4cc`](https://gi

[GitHub] spark pull request: [SQL] Prevent illegal NULL propagation when fi...

2016-05-24 Thread yhuai
GitHub user yhuai opened a pull request: https://github.com/apache/spark/pull/13290 [SQL] Prevent illegal NULL propagation when filtering outer-join results ## What changes were proposed in this pull request? This is another approach for addressing SPARK-13484 (the first approach

[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread wangmiao1981
Github user wangmiao1981 commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221483353 @shivaram @felixcheung For the failure pipedRDD test case, if I copy & paste it in the SparkR, it works fine. --- If your project is set up for it, you can reply

[GitHub] spark pull request: [SPARK-15413] [ML] [MLLIB] Change `toBreeze` t...

2016-05-24 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/13198#issuecomment-221481150 It is more correct and it is a private API. So this LGTM pending Jenkins. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark pull request: [SPARK-15413] [ML] [MLLIB] Change `toBreeze` t...

2016-05-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13198#issuecomment-221481073 **[Test build #59258 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59258/consoleFull)** for PR 13198 at commit [`33485b5`](https://gi

[GitHub] spark pull request: [SPARK-15413] [ML] [MLLIB] Change `toBreeze` t...

2016-05-24 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/13198#issuecomment-221480985 test this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this featu

[GitHub] spark pull request: [SPARK-15364][ML][PySpark] Implement PySpark p...

2016-05-24 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/13219#issuecomment-221480232 @viirya I think we need a test for the picklers. See https://github.com/apache/spark/blob/master/mllib/src/test/scala/org/apache/spark/mllib/api/python/PythonMLLibAPISuit

[GitHub] spark pull request: [SPARK-12922][SparkR][WIP] Implement gapply() ...

2016-05-24 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/12836#issuecomment-221478783 Lets keep it as `dapply` - The specific choice of applying on a partition as a data frame is built into its semantics. If we do build a single row UDF then we can make

[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221478629 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-15008][ML][PySpark] Add integration tes...

2016-05-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12875#issuecomment-221478643 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-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221478607 **[Test build #59256 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59256/consoleFull)** for PR 13284 at commit [`5ac9b9b`](https://g

[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221478631 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-15008][ML][PySpark] Add integration tes...

2016-05-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12875#issuecomment-221478641 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-15008][ML][PySpark] Add integration tes...

2016-05-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12875#issuecomment-221478570 **[Test build #59257 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59257/consoleFull)** for PR 12875 at commit [`218ecba`](https://g

[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221477942 as for subset - maybe tighter signature matching checks in R 3.3.0? I couldn't find any reference on that though. --- If your project is set up for it, you can rep

[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221477791 so startsWith endsWith were definitely added in 3.3.0. I think we should be able to change the generic to match the signature to get it to work (have not tried i

[GitHub] spark pull request: [SPARK-12922][SparkR][WIP] Implement gapply() ...

2016-05-24 Thread sun-rui
Github user sun-rui commented on the pull request: https://github.com/apache/spark/pull/12836#issuecomment-221477758 @NarineK, mapply does not apply here, but we can later add something like lapply, which applies a function to each row in a DataFrame. I suddenly realized that it could

[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221477450 Ok - so that explains one of the problems. Does anybody know what the problem in `subset` is ? --- If your project is set up for it, you can reply to this email and h

[GitHub] spark pull request: [SPARK-15008][ML][PySpark] Add integration tes...

2016-05-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12875#issuecomment-221477146 **[Test build #59257 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59257/consoleFull)** for PR 12875 at commit [`218ecba`](https://gi

[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread wangmiao1981
Github user wangmiao1981 commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221477072 @shivaram I will make the change with R version check. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221476748 Continuing from my previous message, we can't add or remove `endsWith` and `startsWith` as we want to support all R versions from 3.1.0 onwards. We could get the R ver

[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221476475 **[Test build #59256 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59256/consoleFull)** for PR 13284 at commit [`5ac9b9b`](https://gi

[GitHub] spark pull request: [SPARK-13850] Force the sorter to Spill when n...

2016-05-24 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/13107#issuecomment-221476424 TimSort require a temporary buffer to store the shorter part, which could be half of the size of pointer array in worst case. This depends on the original order of rows,

[GitHub] spark pull request: [SPARK-15491][SQL]fix assertion failure for JD...

2016-05-24 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/13287#issuecomment-221476178 Thanks for the pull request. Can you format the description better? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221476116 Hmm looks like `startsWith` and `endsWith` were added in R 3.3.0 - See http://www.r-statistics.com/2016/05/r-3-3-0-is-released/ --- If your project is set up fo

[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread wangmiao1981
Github user wangmiao1981 commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221476011 @shivaram The pipedRDD one seems working when using sudo in Linux. My mac does not work though. --- If your project is set up for it, you can reply to this email

[GitHub] spark pull request: [SPARK-12071][Doc] Document the behaviour of N...

2016-05-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13268 --- 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-15412][PySpark][SparkR][DOCS] Improve l...

2016-05-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13199 --- 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-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread wangmiao1981
Github user wangmiao1981 commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221475769 @felixcheung > conflicts(detail = TRUE) $.GlobalEnv [1] "df" $`package:SparkR` [1] "alias" "cov" "filter""fitte

[GitHub] spark pull request: [SPARK-12071][Doc] Document the behaviour of N...

2016-05-24 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/13268#issuecomment-221475713 LGTM. Merging this to master and branch-2.0 --- 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 pro

[GitHub] spark pull request: [SPARK-15412][PySpark][SparkR][DOCS] Improve l...

2016-05-24 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/13199#issuecomment-221475488 LGTM. Merging this to master and branch-2.0 --- 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 pro

[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread wangmiao1981
Github user wangmiao1981 commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221475397 R version 3.3.0 (2016-05-03) -- "Supposedly Educational" Copyright (C) 2016 The R Foundation for Statistical Computing Platform: x86_64-apple-darwin13.4.0 (6

[GitHub] spark pull request: [SPARK-15008][ML][PySpark] Add integration tes...

2016-05-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12875#issuecomment-221474796 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-15008][ML][PySpark] Add integration tes...

2016-05-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12875#issuecomment-221474798 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-15008][ML][PySpark] Add integration tes...

2016-05-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12875#issuecomment-221474776 **[Test build #59255 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59255/consoleFull)** for PR 12875 at commit [`39e5361`](https://g

[GitHub] spark pull request: [SPARK-15508][Streaming][Tests]Fix flaky test:...

2016-05-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13281 --- 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-15008][ML][PySpark] Add integration tes...

2016-05-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12875#issuecomment-221473671 **[Test build #59255 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59255/consoleFull)** for PR 12875 at commit [`39e5361`](https://gi

[GitHub] spark pull request: [SPARK-15508][Streaming][Tests]Fix flaky test:...

2016-05-24 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/13281#issuecomment-221473561 Thanks for reviewing. Merging to master / 2.0. --- 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 p

[GitHub] spark pull request: [SPARK-15451][build] Use jdk7's rt.jar when av...

2016-05-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13272#issuecomment-221473050 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-15451][build] Use jdk7's rt.jar when av...

2016-05-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13272#issuecomment-221473047 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-15451][build] Use jdk7's rt.jar when av...

2016-05-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13272#issuecomment-221472935 **[Test build #59252 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59252/consoleFull)** for PR 13272 at commit [`636658e`](https://g

[GitHub] spark pull request: [SPARK-14998][SQL]fix ArrayIndexOutOfBoundsExc...

2016-05-24 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/12772#issuecomment-221470057 Sorry, I am unable to reproduce it. Without the fix, the following test case works well. ```scala val data = Seq(("A\tB\tC\tD\t\t"), ("E\tF\tD\tE\t"

[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread felixcheung
Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/13284#discussion_r64513464 --- Diff: R/pkg/R/DataFrame.R --- @@ -1445,7 +1445,11 @@ setMethod("[", signature(x = "SparkDataFrame"), #' } setMethod("subset", signature(x =

[GitHub] spark pull request: [SPARK-15498][TESTS] fix slow tests

2016-05-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13273 --- 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-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread felixcheung
Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/13284#discussion_r64513468 --- Diff: R/pkg/R/DataFrame.R --- @@ -1445,7 +1445,11 @@ setMethod("[", signature(x = "SparkDataFrame"), #' } setMethod("subset", signature(x =

[GitHub] spark pull request: [SPARK-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread felixcheung
Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/13284#discussion_r64513429 --- Diff: R/pkg/R/DataFrame.R --- @@ -1445,7 +1445,11 @@ setMethod("[", signature(x = "SparkDataFrame"), #' } setMethod("subset", signature(x =

[GitHub] spark pull request: [SPARK-15498][TESTS] fix slow tests

2016-05-24 Thread liancheng
Github user liancheng commented on the pull request: https://github.com/apache/spark/pull/13273#issuecomment-221469762 Merging to master and branch-2.0. --- 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-15439][SparkR]:Failed to run unit test ...

2016-05-24 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/spark/pull/13284#issuecomment-221469622 I'm seeing the error with `subset` as well. As for "endsWith", "startsWith" - could you check what you have in your environment - you could run `conflicts(detail

[GitHub] spark pull request: [SPARK-15493][SQL] Allow setting the quoteEsca...

2016-05-24 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/13267#issuecomment-221469255 Yea I agree with escapeQuotes. --- 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 ha

[GitHub] spark pull request: [SPARK-15493][SQL] Allow setting the quoteEsca...

2016-05-24 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/13267#issuecomment-221469276 @jurriaan want to do the change? --- 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-15493][SQL] Allow setting the quoteEsca...

2016-05-24 Thread falaki
Github user falaki commented on the pull request: https://github.com/apache/spark/pull/13267#issuecomment-221468338 @rxin and @jurriaan I agree to keep it set by default. However, I think it is better to leave it configurable. In two cases before, I assumed a reasonable default value

[GitHub] spark pull request: [SPARK-15365] [SQL]: When table size statistic...

2016-05-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13150 --- 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-15365] [SQL]: When table size statistic...

2016-05-24 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/13150#issuecomment-221467220 Merging in master/2.0. --- 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-15365] [SQL]: When table size statistic...

2016-05-24 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/13150#issuecomment-221467315 @Parth-Brahmbhatt you should add the email address you used in your commit to your github profile, so the commit is associated with your account. Thanks. --- If your proj

[GitHub] spark pull request: [SPARK-15518] Rename various scheduler backend...

2016-05-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13288 --- 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-15518] Rename various scheduler backend...

2016-05-24 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/13288#issuecomment-221466892 Thanks - merging in master/2.0. --- 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 h

[GitHub] spark pull request: [WIP] [SPARK-10903] [SPARKR] R - Simplify SQLC...

2016-05-24 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/9192#issuecomment-221466855 Thanks - look forward to 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-15515] [SPARK-15514] [SQL] Error Handli...

2016-05-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13283#issuecomment-221466629 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-15515] [SPARK-15514] [SQL] Error Handli...

2016-05-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13283#issuecomment-221466626 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-15515] [SPARK-15514] [SQL] Error Handli...

2016-05-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13283#issuecomment-221466522 **[Test build #59249 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59249/consoleFull)** for PR 13283 at commit [`f6a4517`](https://g

[GitHub] spark pull request: [WIP] [SPARK-10903] [SPARKR] R - Simplify SQLC...

2016-05-24 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/spark/pull/9192#issuecomment-221466080 @rxin - [ ] fix failed tests (hiveContext related) - [ ] check on roxygen2 generated doc - [ ] update code examples in doc to remove sqlContext (I'd like

[GitHub] spark pull request: [SPARK-15515] [SPARK-15514] [SQL] Error Handli...

2016-05-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13283#issuecomment-221466099 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-15515] [SPARK-15514] [SQL] Error Handli...

2016-05-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13283#issuecomment-221466102 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-15515] [SPARK-15514] [SQL] Error Handli...

2016-05-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13283#issuecomment-221466000 **[Test build #59254 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59254/consoleFull)** for PR 13283 at commit [`c313ffc`](https://g

[GitHub] spark pull request: [SPARK-15495][SQL][WIP] Improve the explain ou...

2016-05-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13271#issuecomment-221465283 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-15495][SQL][WIP] Improve the explain ou...

2016-05-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13271#issuecomment-221465280 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-15495][SQL][WIP] Improve the explain ou...

2016-05-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/13271#issuecomment-221465200 **[Test build #59250 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59250/consoleFull)** for PR 13271 at commit [`8d507bf`](https://g

[GitHub] spark pull request: [SPARK-15517][SQL][STREAMING] Add support for ...

2016-05-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13286#issuecomment-221464878 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-15517][SQL][STREAMING] Add support for ...

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

  1   2   3   4   5   6   7   >