[GitHub] spark pull request: [Minor][DOC] Minor typo fixes

2016-04-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12755 --- 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][DOC] Minor typo fixes

2016-04-28 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12755#issuecomment-215632671 Merging in master. 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 have

[GitHub] spark pull request: [Minor][DOC] Minor typo fixes

2016-04-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12755#issuecomment-215632083 **[Test build #2930 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2930/consoleFull)** for PR 12755 at commit

[GitHub] spark pull request: [Minor][DOC] Minor typo fixes

2016-04-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12755#issuecomment-215623845 **[Test build #2930 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2930/consoleFull)** for PR 12755 at commit

[GitHub] spark pull request: [Minor][DOC] Minor typo fixes

2016-04-28 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12755#issuecomment-215623807 LGTM pending jenkins. --- 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][DOC] Minor typo fixes

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

[GitHub] spark pull request: [Minor][DOC] Minor typo fixes

2016-04-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12755#issuecomment-215617449 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: [Minor][DOC] Minor typo fixes

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

[GitHub] spark pull request: [Minor][DOC] Minor typo fixes

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

[GitHub] spark pull request: [Minor][DOC] Minor typo fixes

2016-04-28 Thread zhengruifeng
Github user zhengruifeng commented on the pull request: https://github.com/apache/spark/pull/12755#issuecomment-215610516 @rxin The PR was first created to fix errors in Dataset.scala doc. What about updating this PR by remove changes in other files? --- If your project is set up

[GitHub] spark pull request: [Minor][DOC] Minor typo fixes

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

[GitHub] spark pull request: [Minor][DOC] Minor typo fixes

2016-04-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12755#issuecomment-215496944 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: [Minor][DOC] Minor typo fixes

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

[GitHub] spark pull request: [Minor][DOC] Minor typo fixes

2016-04-28 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12755#issuecomment-215495090 @zhengruifeng Thanks. While the double space isn't ideal, it isn't wrong either. Merging this right before the code freeze can cause a lot of merge conflicts with other

[GitHub] spark pull request: [Minor][DOC] Minor typo fixes

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

[GitHub] spark pull request: [Minor][DOC] Minor typo fixes

2016-04-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12755#issuecomment-215486833 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: [Minor][DOC] Minor typo fixes

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

[GitHub] spark pull request: [Minor][DOC] Minor typo fixes

2016-04-28 Thread zhengruifeng
Github user zhengruifeng commented on the pull request: https://github.com/apache/spark/pull/12755#issuecomment-215471995 Jenkins 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

[GitHub] spark pull request: [Minor][DOC] Minor typo fixes

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

[GitHub] spark pull request: [Minor][DOC] Minor typo fixes

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

[GitHub] spark pull request: [Minor][DOC] Minor typo fixes

2016-04-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12755#issuecomment-215466200 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: [Minor][DOC] Minor typo fixes

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

[GitHub] spark pull request: [Minor][DOC] Minor typo fixes in Dataset.scala

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

[GitHub] spark pull request: [Minor][DOC] Minor typo fixes in Dataset.scala

2016-04-28 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/12755#issuecomment-215439210 OK, that's pretty trivial. Can you take a pass over other related files for typos and formatting problems? --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: [Minor][DOC] Minor typo fixes in Dataset.scala

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

[GitHub] spark pull request: [Minor][DOC] Minor typo fixes in Dataset.scala

2016-04-28 Thread zhengruifeng
GitHub user zhengruifeng opened a pull request: https://github.com/apache/spark/pull/12755 [Minor][DOC] Minor typo fixes in Dataset.scala ## What changes were proposed in this pull request? Minor typo fixes ## How was this patch tested? local build You can merge