[GitHub] spark pull request: [DOC] Adjust coverage for partitionBy()

2016-01-04 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/10499#issuecomment-168800225 Thanks, merged to master and 1.6 --- 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: [DOC] Adjust coverage for partitionBy()

2016-01-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10499 --- 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: [DOC] Adjust coverage for partitionBy()

2016-01-04 Thread tedyu
Github user tedyu commented on the pull request: https://github.com/apache/spark/pull/10499#issuecomment-168756031 @marmbrus Gentle ping --- 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: [DOC] Adjust coverage for partitionBy()

2015-12-30 Thread tedyu
Github user tedyu commented on the pull request: https://github.com/apache/spark/pull/10499#issuecomment-168059853 The test failure was not related to the patch. Looks like HiveThriftBinaryServerSuite timed out: ``` [info] HiveThriftBinaryServerSuite: [info] - GetInfo

[GitHub] spark pull request: [DOC] Adjust coverage for partitionBy()

2015-12-30 Thread tedyu
Github user tedyu commented on the pull request: https://github.com/apache/spark/pull/10499#issuecomment-168059897 @marmbrus : Is there anything I need to do ? --- 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: [DOC] Adjust coverage for partitionBy()

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

[GitHub] spark pull request: [DOC] Adjust coverage for partitionBy()

2015-12-29 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/10499#discussion_r48565802 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala --- @@ -119,7 +119,7 @@ final class DataFrameWriter private[sql](df:

[GitHub] spark pull request: [DOC] Adjust coverage for partitionBy()

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

[GitHub] spark pull request: [DOC] Adjust coverage for partitionBy()

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

[GitHub] spark pull request: [DOC] Adjust coverage for partitionBy()

2015-12-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10499#issuecomment-167945033 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: [DOC] Adjust coverage for partitionBy()

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

[GitHub] spark pull request: [DOC] Adjust coverage for partitionBy()

2015-12-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10499#issuecomment-167697956 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: [DOC] Adjust coverage for partitionBy()

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

[GitHub] spark pull request: [DOC] Adjust coverage for partitionBy()

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

[GitHub] spark pull request: [DOC] Adjust coverage for partitionBy()

2015-12-28 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/10499#discussion_r48518567 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala --- @@ -119,7 +119,7 @@ final class DataFrameWriter private[sql](df:

[GitHub] spark pull request: [DOC] Adjust coverage for partitionBy()

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

[GitHub] spark pull request: [DOC] Adjust coverage for partitionBy()

2015-12-28 Thread ted-yu
GitHub user ted-yu opened a pull request: https://github.com/apache/spark/pull/10499 [DOC] Adjust coverage for partitionBy() This is the related thread: http://search-hadoop.com/m/q3RTtO3ReeJ1iF02=Re+partitioning+json+data+in+spark Michael suggested fixing the doc.

[GitHub] spark pull request: [DOC] Adjust coverage for partitionBy()

2015-12-28 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/10499#discussion_r48508973 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala --- @@ -119,7 +119,7 @@ final class DataFrameWriter private[sql](df:

[GitHub] spark pull request: [DOC] Adjust coverage for partitionBy()

2015-12-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10499#issuecomment-167670012 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: [DOC] Adjust coverage for partitionBy()

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

[GitHub] spark pull request: [DOC] Adjust coverage for partitionBy()

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

[GitHub] spark pull request: [DOC] Adjust coverage for partitionBy()

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

[GitHub] spark pull request: [DOC] Adjust coverage for partitionBy()

2015-12-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10499#issuecomment-167715530 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: [DOC] Adjust coverage for partitionBy()

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

[GitHub] spark pull request: [DOC] Adjust coverage for partitionBy()

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