[GitHub] spark issue #14478: [SPARK-16875][SQL] Add args checking for DataSet randomS...

2016-08-04 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14478 Merged to 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 feature enabled and wishes

[GitHub] spark issue #14478: [SPARK-16875][SQL] Add args checking for DataSet randomS...

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

[GitHub] spark issue #14478: [SPARK-16875][SQL] Add args checking for DataSet randomS...

2016-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14478 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

[GitHub] spark issue #14478: [SPARK-16875][SQL] Add args checking for DataSet randomS...

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

[GitHub] spark issue #14478: [SPARK-16875][SQL] Add args checking for DataSet randomS...

2016-08-03 Thread zhengruifeng
Github user zhengruifeng commented on the issue: https://github.com/apache/spark/pull/14478 @srowen Right. I just add those checking for RDD. --- 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 issue #14478: [SPARK-16875][SQL] Add args checking for DataSet randomS...

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

[GitHub] spark issue #14478: [SPARK-16875][SQL] Add args checking for DataSet randomS...

2016-08-03 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14478 Ah, let's do this for the RDD API too eh? --- 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

[GitHub] spark issue #14478: [SPARK-16875][SQL] Add args checking for DataSet randomS...

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

[GitHub] spark issue #14478: [SPARK-16875][SQL] Add args checking for DataSet randomS...

2016-08-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14478 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

[GitHub] spark issue #14478: [SPARK-16875][SQL] Add args checking for DataSet randomS...

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

[GitHub] spark issue #14478: [SPARK-16875][SQL] Add args checking for DataSet randomS...

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

[GitHub] spark issue #14478: [SPARK-16875][SQL] Add args checking for DataSet randomS...

2016-08-03 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/14478 LGTM. I wonder if we should even disallow a 0% sample, but, OK as-is. --- 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