[GitHub] spark pull request: [TRIVIAL][Doc] SparkSession class doc example ...

2016-05-13 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/13086#issuecomment-219081043
  
**[Test build #58579 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58579/consoleFull)**
 for PR 13086 at commit 
[`7000168`](https://github.com/apache/spark/commit/70001680da48df956e2a69a70b7ac14f109e0676).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [TRIVIAL][Doc] SparkSession class doc example ...

2016-05-13 Thread tejasapatil
Github user tejasapatil commented on the pull request:

https://github.com/apache/spark/pull/13086#issuecomment-219080926
  
@rxin : Thanks for review. I have made the suggested change.

```
scala> import org.apache.spark.sql._
import org.apache.spark.sql._

scala> SparkSession.builder().master("local").appName("Word 
Count").getOrCreate()
res0: org.apache.spark.sql.SparkSession = 
org.apache.spark.sql.SparkSession@65c17e38

scala> SparkSession.builder.master("local").appName("Word 
Count").getOrCreate()
res1: org.apache.spark.sql.SparkSession = 
org.apache.spark.sql.SparkSession@65c17e38
```


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [TRIVIAL][Doc] SparkSession class doc example ...

2016-05-12 Thread rxin
Github user rxin commented on the pull request:

https://github.com/apache/spark/pull/13086#issuecomment-218950905
  
Maybe we should add () to the builder function instead? In that case this 
code will work in Java 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 does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [TRIVIAL][Doc] SparkSession class doc example ...

2016-05-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/13086#issuecomment-218930724
  
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [TRIVIAL][Doc] SparkSession class doc example ...

2016-05-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/13086#issuecomment-218930725
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58526/
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 and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [TRIVIAL][Doc] SparkSession class doc example ...

2016-05-12 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/13086#issuecomment-218930610
  
**[Test build #58526 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58526/consoleFull)**
 for PR 13086 at commit 
[`5678bb9`](https://github.com/apache/spark/commit/5678bb91d006e58a9ac163330bc72058d3d3cce5).
 * This patch passes all tests.
 * This patch merges cleanly.
 * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [TRIVIAL][Doc] SparkSession class doc example ...

2016-05-12 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request:

https://github.com/apache/spark/pull/13086#issuecomment-218922187
  
(Just to make sure, it seems it is only single one across Spark codes)

```bash
grep -r "SparkSession.builder()" . | grep ".scala"
./sql/core/src/main/scala/org/apache/spark/sql/SparkSession.scala: *   
SparkSession.builder()
```


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [TRIVIAL][Doc] SparkSession class doc example ...

2016-05-12 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/13086#issuecomment-218920811
  
**[Test build #58526 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58526/consoleFull)**
 for PR 13086 at commit 
[`5678bb9`](https://github.com/apache/spark/commit/5678bb91d006e58a9ac163330bc72058d3d3cce5).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [TRIVIAL][Doc] SparkSession class doc example ...

2016-05-12 Thread tejasapatil
GitHub user tejasapatil opened a pull request:

https://github.com/apache/spark/pull/13086

[TRIVIAL][Doc] SparkSession class doc example correction

## What changes were proposed in this pull request?

Was trying out `SparkSession` for the first time and the given class doc 
(when copied as is) did not work over Spark shell:

```
scala> import org.apache.spark.sql._
import org.apache.spark.sql._

scala> SparkSession.builder().master("local").appName("Word 
Count").getOrCreate()
:27: error: org.apache.spark.sql.SparkSession.Builder does not 
take parameters
   SparkSession.builder().master("local").appName("Word 
Count").getOrCreate()
```

PS: I am using latest master

## How was this patch tested?

```
scala> SparkSession.builder.master("local").appName("Word 
Count").getOrCreate()
res0: org.apache.spark.sql.SparkSession = 
org.apache.spark.sql.SparkSession@c3719e5
```

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/tejasapatil/spark doc_correction

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/spark/pull/13086.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #13086


commit 5678bb91d006e58a9ac163330bc72058d3d3cce5
Author: Tejas Patil 
Date:   2016-05-13T00:02:44Z

SparkSession class doc correction




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org