[GitHub] spark pull request: [MINOR][SQL][DOCS] Fix docs of Dataset.scala and SQLImpl...

2016-05-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13420 --- 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: [MINOR][SQL][DOCS] Fix docs of Dataset.scala and SQLImpl...

2016-05-31 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/13420 Thank you, @srowen and @andrewor14 ! --- 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: [MINOR][SQL][DOCS] Fix docs of Dataset.scala and SQLImpl...

2016-05-31 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/13420#discussion_r65286389 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala --- @@ -93,7 +93,7 @@ private[sql] object Dataset { * to some files on

[GitHub] spark pull request: [MINOR][SQL][DOCS] Fix docs of Dataset.scala and SQLImpl...

2016-05-31 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/13420 LGTM I'll add the semicolon 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: [MINOR][SQL][DOCS] Fix docs of Dataset.scala and SQLImpl...

2016-05-31 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/13420#discussion_r65285531 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala --- @@ -93,7 +93,7 @@ private[sql] object Dataset { * to some files on storage

[GitHub] spark pull request: [MINOR][SQL][DOCS] Fix docs of Dataset.scala and SQLImpl...

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

[GitHub] spark pull request: [MINOR][SQL][DOCS] Fix docs of Dataset.scala and SQLImpl...

2016-05-31 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13420 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

[GitHub] spark pull request: [MINOR][SQL][DOCS] Fix docs of Dataset.scala and SQLImpl...

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

[GitHub] spark pull request: [MINOR][SQL][DOCS] Fix docs of Dataset.scala and SQLImpl...

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

[GitHub] spark pull request: [MINOR][SQL][DOCS] Fix docs of Dataset.scala and SQLImpl...

2016-05-31 Thread dongjoon-hyun
GitHub user dongjoon-hyun opened a pull request: https://github.com/apache/spark/pull/13420 [MINOR][SQL][DOCS] Fix docs of Dataset.scala and SQLImplicits.scala. ## What changes were proposed in this pull request? This PR fixes a sample code, a description, and indentations