[GitHub] spark issue #18379: [SPARK-21164] [SQL] Remove isTableSample from Sample and...

2017-06-23 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/18379 Thanks! Merging to master. --- 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

[GitHub] spark issue #18379: [SPARK-21164] [SQL] Remove isTableSample from Sample and...

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

[GitHub] spark issue #18379: [SPARK-21164] [SQL] Remove isTableSample from Sample and...

2017-06-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18379 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 #18379: [SPARK-21164] [SQL] Remove isTableSample from Sample and...

2017-06-23 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18379 **[Test build #78536 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78536/testReport)** for PR 18379 at commit

[GitHub] spark issue #18379: [SPARK-21164] [SQL] Remove isTableSample from Sample and...

2017-06-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18379 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/78535/ Test FAILed. ---

[GitHub] spark issue #18379: [SPARK-21164] [SQL] Remove isTableSample from Sample and...

2017-06-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18379 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 project does not have this feature

[GitHub] spark issue #18379: [SPARK-21164] [SQL] Remove isTableSample from Sample and...

2017-06-23 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18379 **[Test build #78535 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78535/testReport)** for PR 18379 at commit

[GitHub] spark issue #18379: [SPARK-21164] [SQL] Remove isTableSample from Sample and...

2017-06-23 Thread hvanhovell
Github user hvanhovell commented on the issue: https://github.com/apache/spark/pull/18379 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 feature enabled and

[GitHub] spark issue #18379: [SPARK-21164] [SQL] Remove isTableSample from Sample and...

2017-06-23 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18379 **[Test build #78536 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78536/testReport)** for PR 18379 at commit

[GitHub] spark issue #18379: [SPARK-21164] [SQL] Remove isTableSample from Sample and...

2017-06-23 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18379 **[Test build #78535 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78535/testReport)** for PR 18379 at commit

[GitHub] spark issue #18379: [SPARK-21164] [SQL] Remove isTableSample from Sample

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

[GitHub] spark issue #18379: [SPARK-21164] [SQL] Remove isTableSample from Sample

2017-06-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18379 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 #18379: [SPARK-21164] [SQL] Remove isTableSample from Sample

2017-06-21 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18379 **[Test build #78402 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78402/testReport)** for PR 18379 at commit

[GitHub] spark issue #18379: [SPARK-21164] [SQL] Remove isTableSample from Sample

2017-06-21 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18379 **[Test build #78402 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78402/testReport)** for PR 18379 at commit

[GitHub] spark issue #18379: [SPARK-21164] [SQL] Remove isTableSample from Sample

2017-06-21 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/18379 @hvanhovell Yes! Let me remove them in this PR 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

[GitHub] spark issue #18379: [SPARK-21164] [SQL] Remove isTableSample from Sample

2017-06-21 Thread hvanhovell
Github user hvanhovell commented on the issue: https://github.com/apache/spark/pull/18379 Shouldn't we remove all the `isGenerated` stuff then? See `Alias` for example. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark issue #18379: [SPARK-21164] [SQL] Remove isTableSample from Sample

2017-06-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18379 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/78399/ Test FAILed. ---

[GitHub] spark issue #18379: [SPARK-21164] [SQL] Remove isTableSample from Sample

2017-06-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18379 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 project does not have this feature

[GitHub] spark issue #18379: [SPARK-21164] [SQL] Remove isTableSample from Sample

2017-06-21 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18379 **[Test build #78399 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78399/testReport)** for PR 18379 at commit

[GitHub] spark issue #18379: [SPARK-21164] [SQL] Remove isTableSample from Sample

2017-06-21 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18379 **[Test build #78399 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/78399/testReport)** for PR 18379 at commit