[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-17 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/13635 Oh, great!! --- 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

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-17 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/spark/pull/13635 awesome! --- 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 t

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-17 Thread shivaram
Github user shivaram commented on the issue: https://github.com/apache/spark/pull/13635 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 project does not have this featur

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-17 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/spark/pull/13635 Test passed and merge cleanly. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13635 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60735/ Test PASSed. ---

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13635 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 not have this feature enabled

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-17 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13635 **[Test build #60735 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60735/consoleFull)** for PR 13635 at commit [`b6b8712`](https://github.com/apache/spark/commit/

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13635 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 project does not have this feature e

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13635 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60738/ Test PASSed. ---

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-17 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13635 **[Test build #60738 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60738/consoleFull)** for PR 13635 at commit [`4bc5449`](https://github.com/apache/spark/commit/

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-17 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13635 **[Test build #60738 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60738/consoleFull)** for PR 13635 at commit [`4bc5449`](https://github.com/apache/spark/commit/4

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-17 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/spark/pull/13635 Oops. 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 feature enabled and wishes so, or i

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-17 Thread shivaram
Github user shivaram commented on the issue: https://github.com/apache/spark/pull/13635 Thanks - Could you also bring this up to date with master branch ? --- 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 d

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-17 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13635 **[Test build #60735 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60735/consoleFull)** for PR 13635 at commit [`b6b8712`](https://github.com/apache/spark/commit/b

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-17 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/spark/pull/13635 Done. Thanks ``` Welcome to __ / __/__ ___ _/ /__ _\ \/ _ \/ _ `/ __/ '_/ /___/ .__/\_,_/_/ /_/\_\ version 2.0.0-SNAPSHOT

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-17 Thread shivaram
Github user shivaram commented on the issue: https://github.com/apache/spark/pull/13635 @felixcheung Thanks for the update. The change looks pretty good to me. I think there are 2-3 follow up JIRAs I opened from the review that can have separate PRs. There was only one comment in she

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13635 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60688/ Test PASSed. ---

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13635 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 project does not have this feature e

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-17 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13635 **[Test build #60688 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60688/consoleFull)** for PR 13635 at commit [`2ed0788`](https://github.com/apache/spark/commit/

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13635 **[Test build #60688 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60688/consoleFull)** for PR 13635 at commit [`2ed0788`](https://github.com/apache/spark/commit/2

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-16 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/spark/pull/13635 updated per feedback + more tests --- 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 enabl

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-16 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/spark/pull/13635 I guess I also need to merge with this https://github.com/apache/spark/pull/13714/files#r67438295 --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-15 Thread sun-rui
Github user sun-rui commented on the issue: https://github.com/apache/spark/pull/13635 @shivaram, I probably take a look at this tonight. --- 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 issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-15 Thread shivaram
Github user shivaram commented on the issue: https://github.com/apache/spark/pull/13635 Thanks @felixcheung for the PR. Other than the naming issues, I think the code changes look pretty good to me. I think there are some more docs, programming guide changes we'll need to make but I

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-13 Thread shivaram
Github user shivaram commented on the issue: https://github.com/apache/spark/pull/13635 Thanks @felixcheung - I'll take a look at this today. cc @rxin --- 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 issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13635 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60388/ Test PASSed. ---

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13635 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 project does not have this feature e

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-13 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13635 **[Test build #60388 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60388/consoleFull)** for PR 13635 at commit [`107dbef`](https://github.com/apache/spark/commit/

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13635 **[Test build #60388 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60388/consoleFull)** for PR 13635 at commit [`107dbef`](https://github.com/apache/spark/commit/1

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13635 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 project does not have this feature e

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13635 **[Test build #60385 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60385/consoleFull)** for PR 13635 at commit [`a48b7ea`](https://github.com/apache/spark/commit/

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13635 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60385/ Test FAILed. ---

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13635 **[Test build #60385 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60385/consoleFull)** for PR 13635 at commit [`a48b7ea`](https://github.com/apache/spark/commit/a

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13635 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 project does not have this feature e

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13635 **[Test build #60384 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60384/consoleFull)** for PR 13635 at commit [`b494232`](https://github.com/apache/spark/commit/

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/13635 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60384/ Test FAILed. ---

[GitHub] spark issue #13635: [SPARK-15159][SPARKR] SparkR SparkSession API

2016-06-12 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/13635 **[Test build #60384 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60384/consoleFull)** for PR 13635 at commit [`b494232`](https://github.com/apache/spark/commit/b