[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2016-01-15 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-171993499 LGTM from me as well. Thanks @olarayej - Merging this to master and branch-1.6. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2016-01-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/9613 --- 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-11031][SPARKR] Method str() on a DataFr...

2016-01-15 Thread olarayej
Github user olarayej commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-172054570 Thanks, @shivaram! --- 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-11031][SPARKR] Method str() on a DataFr...

2016-01-13 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-171409820 **[Test build #49327 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49327/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2016-01-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-171416099 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2016-01-13 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-171415929 **[Test build #49327 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49327/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2016-01-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-171416101 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2016-01-13 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-171473356 looks good and 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 project does not

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2016-01-13 Thread olarayej
Github user olarayej commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-171441589 @felixcheung Done. Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2016-01-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-171011101 **[Test build #49249 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49249/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2016-01-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-17109 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2016-01-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-171010984 **[Test build #49249 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49249/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2016-01-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-171010412 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2016-01-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-171010408 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2016-01-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-17106 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2016-01-12 Thread olarayej
Github user olarayej commented on a diff in the pull request: https://github.com/apache/spark/pull/9613#discussion_r49493628 --- Diff: R/pkg/R/generics.R --- @@ -581,6 +579,10 @@ setGeneric("unionAll", function(x, y) { standardGeneric("unionAll") }) #' @export

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2016-01-12 Thread olarayej
Github user olarayej commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-171013162 Jenkins, could you retest please? The error I see is "Error fetching remote repo 'origin'" --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2016-01-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-171109422 **[Test build #49272 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49272/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2016-01-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-171109426 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2016-01-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-171109334 **[Test build #49272 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49272/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2016-01-12 Thread olarayej
Github user olarayej commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-171101731 @SparkQA Could you retest? --- 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-11031][SPARKR] Method str() on a DataFr...

2016-01-12 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-171106630 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

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2016-01-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-171109431 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2016-01-11 Thread felixcheung
Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/9613#discussion_r49412620 --- Diff: R/pkg/R/generics.R --- @@ -581,6 +579,10 @@ setGeneric("unionAll", function(x, y) { standardGeneric("unionAll") }) #' @export

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2016-01-11 Thread felixcheung
Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/9613#discussion_r49412666 --- Diff: R/pkg/R/types.R --- @@ -47,6 +47,19 @@ COMPLEX_TYPES <- list( # The full list of data types. DATA_TYPES <-

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2016-01-11 Thread olarayej
Github user olarayej commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-170624621 Happy New Year, folks! Shall we merge this PR? @shivaram --- 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-11031][SPARKR] Method str() on a DataFr...

2016-01-11 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-170676034 Sorry I've been traveling for the last couple of weeks. I get back on Tuesday and will take a look On Jan 11, 2016 09:23, "Oscar D. Lara Yejas"

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-21 Thread olarayej
Github user olarayej commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-166544706 @shivaram I have addressed all your comments. Should we close this pull request? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165877280 **[Test build #48016 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48016/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165891723 **[Test build #48021 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48021/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

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

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165876608 **[Test build #48016 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48016/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165891730 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165877284 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

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

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

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

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165891159 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165891346 **[Test build #48021 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48021/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-18 Thread olarayej
Github user olarayej commented on a diff in the pull request: https://github.com/apache/spark/pull/9613#discussion_r48079567 --- Diff: R/pkg/R/generics.R --- @@ -509,13 +520,8 @@ setGeneric("saveAsTable", function(df, tableName, source, mode, ...) {

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-18 Thread olarayej
Github user olarayej commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165923895 @felixcheung 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

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165913264 **[Test build #48028 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48028/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165913380 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

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

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-18 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165909818 **[Test build #48028 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48028/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165562671 **[Test build #47945 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47945/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165568631 **[Test build #47945 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47945/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

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

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165568696 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165583311 **[Test build #47950 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47950/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165597087 **[Test build #47957 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47957/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165589014 **[Test build #47950 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47950/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165589082 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

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

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

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

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-17 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165619311 I think @shivaram had a few more comments you might want to address: https://github.com/apache/spark/pull/9613#discussion_r45796120 --- If your project is set up

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165606115 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-17 Thread olarayej
Github user olarayej commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165607267 @shivaram I have removed the caching logic as you indicated @felixcheung @sun-rui I have already explained why we can't use R's str() function under the covers.

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165605875 **[Test build #47957 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47957/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-17 Thread olarayej
Github user olarayej commented on a diff in the pull request: https://github.com/apache/spark/pull/9613#discussion_r47984121 --- Diff: R/pkg/R/DataFrame.R --- @@ -2151,3 +2151,97 @@ setMethod("coltypes", rTypes }) + +#' Display the

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165631073 **[Test build #47967 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47967/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165631437 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

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

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165631432 **[Test build #47967 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47967/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-17 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-165669971 also this comment https://github.com/apache/spark/pull/9613#discussion-diff-45796256 --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-162982218 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

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

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-162981999 **[Test build #47349 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47349/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-162975173 **[Test build #47349 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47349/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-08 Thread olarayej
Github user olarayej commented on a diff in the pull request: https://github.com/apache/spark/pull/9613#discussion_r46995766 --- Diff: R/pkg/R/DataFrame.R --- @@ -2151,3 +2151,97 @@ setMethod("coltypes", rTypes }) + +#' Display the

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-162681519 **[Test build #47288 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47288/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-162688181 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-162688103 **[Test build #47288 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47288/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

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

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-07 Thread olarayej
Github user olarayej commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-162725183 @felixcheung, @sun-rui As I mentioned in my previous comment, it's not only replacing data.frame for DataFrame in the header. There are also issues with the number of

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-162732631 **[Test build #47297 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47297/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-162735613 **[Test build #47297 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47297/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

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

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-01 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-161088966 should we simply gsup to replace the substring `data.frame` from `str()` output? It looks like we are already `head()/take()/collect()` a small part of the

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-01 Thread sun-rui
Github user sun-rui commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-160909602 Instead of simulating str() behaivor in our own code, is it possible to collect a subset of DataFrame as a local data.frame, and then call str() on it? This seems easy

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-12-01 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-161022979 @sun-rui We discussed this at https://github.com/apache/spark/pull/9613#discussion_r45532294 --- If your project is set up for it, you can reply to this email and

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-11-24 Thread shivaram
Github user shivaram commented on a diff in the pull request: https://github.com/apache/spark/pull/9613#discussion_r45796625 --- Diff: R/pkg/R/DataFrame.R --- @@ -2151,3 +2151,97 @@ setMethod("coltypes", rTypes }) + +#' Display the

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-11-24 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/9613#discussion_r45798353 --- Diff: R/pkg/R/DataFrame.R --- @@ -2199,3 +2199,97 @@ setMethod("coltypes", rTypes }) + +#' Display the

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-11-24 Thread shivaram
Github user shivaram commented on a diff in the pull request: https://github.com/apache/spark/pull/9613#discussion_r45793856 --- Diff: R/pkg/R/DataFrame.R --- @@ -2199,3 +2199,97 @@ setMethod("coltypes", rTypes }) + +#' Display the

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-11-24 Thread shivaram
Github user shivaram commented on a diff in the pull request: https://github.com/apache/spark/pull/9613#discussion_r45796120 --- Diff: R/pkg/R/DataFrame.R --- @@ -2151,3 +2151,97 @@ setMethod("coltypes", rTypes }) + +#' Display the

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-11-24 Thread shivaram
Github user shivaram commented on a diff in the pull request: https://github.com/apache/spark/pull/9613#discussion_r45796525 --- Diff: R/pkg/R/DataFrame.R --- @@ -2151,3 +2151,97 @@ setMethod("coltypes", rTypes }) + +#' Display the

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-11-24 Thread shivaram
Github user shivaram commented on a diff in the pull request: https://github.com/apache/spark/pull/9613#discussion_r45796256 --- Diff: R/pkg/R/generics.R --- @@ -509,13 +520,8 @@ setGeneric("saveAsTable", function(df, tableName, source, mode, ...) {

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-11-24 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/9613#discussion_r45794118 --- Diff: R/pkg/R/DataFrame.R --- @@ -2199,3 +2199,97 @@ setMethod("coltypes", rTypes }) + +#' Display the

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-11-24 Thread olarayej
Github user olarayej commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-159366139 @shivaram Any further comments or clarification on the existing ones required from my end? Otherwise, should we merge this PR? Thank you! --- If your project is set

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-11-24 Thread sun-rui
Github user sun-rui commented on a diff in the pull request: https://github.com/apache/spark/pull/9613#discussion_r45826251 --- Diff: R/pkg/R/DataFrame.R --- @@ -2199,3 +2199,97 @@ setMethod("coltypes", rTypes }) + +#' Display the

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-11-24 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/9613#discussion_r45826394 --- Diff: R/pkg/R/DataFrame.R --- @@ -2199,3 +2199,97 @@ setMethod("coltypes", rTypes }) + +#' Display the

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-11-24 Thread sun-rui
Github user sun-rui commented on a diff in the pull request: https://github.com/apache/spark/pull/9613#discussion_r45827852 --- Diff: R/pkg/R/DataFrame.R --- @@ -2199,3 +2199,97 @@ setMethod("coltypes", rTypes }) + +#' Display the

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-11-24 Thread felixcheung
Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/9613#discussion_r45829846 --- Diff: R/pkg/R/DataFrame.R --- @@ -2151,3 +2151,97 @@ setMethod("coltypes", rTypes }) + +#' Display the

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-11-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-159035448 **[Test build #46544 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46544/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-11-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-159051454 **[Test build #46548 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46548/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-11-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-159057568 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-11-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-159057394 **[Test build #46548 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46548/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

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

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-11-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-159088695 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 project does

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

2015-11-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9613#issuecomment-159088540 **[Test build #46544 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46544/consoleFull)** for PR 9613 at commit

[GitHub] spark pull request: [SPARK-11031][SPARKR] Method str() on a DataFr...

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

  1   2   3   >