[GitHub] spark issue #16621: [SPARK-19265][SQL] make table relation cache general and...

2017-01-19 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/16621 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 #16621: [SPARK-19265][SQL] make table relation cache general and...

2017-01-19 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/16621 LGTM --- 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

[GitHub] spark issue #16621: [SPARK-19265][SQL] make table relation cache general and...

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

[GitHub] spark issue #16621: [SPARK-19265][SQL] make table relation cache general and...

2017-01-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16621 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 #16621: [SPARK-19265][SQL] make table relation cache general and...

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

[GitHub] spark issue #16621: [SPARK-19265][SQL] make table relation cache general and...

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

[GitHub] spark issue #16621: [SPARK-19265][SQL] make table relation cache general and...

2017-01-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16621 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 #16621: [SPARK-19265][SQL] make table relation cache general and...

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

[GitHub] spark issue #16621: [SPARK-19265][SQL] make table relation cache general and...

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

[GitHub] spark issue #16621: [SPARK-19265][SQL] make table relation cache general and...

2017-01-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16621 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 #16621: [SPARK-19265][SQL] make table relation cache general and...

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

[GitHub] spark issue #16621: [SPARK-19265][SQL] make table relation cache general and...

2017-01-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16621 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 #16621: [SPARK-19265][SQL] make table relation cache general and...

2017-01-17 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/16621 After merging https://github.com/apache/spark/pull/16517, it introduces a few conflicts. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark issue #16621: [SPARK-19265][SQL] make table relation cache general and...

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

[GitHub] spark issue #16621: [SPARK-19265][SQL] make table relation cache general and...

2017-01-17 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/16621 No more comments. It looks pretty good! Let us see whether all the test cases can pass. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark issue #16621: [SPARK-19265][SQL] make table relation cache general and...

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

[GitHub] spark issue #16621: [SPARK-19265][SQL] make table relation cache general and...

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

[GitHub] spark issue #16621: [SPARK-19265][SQL] make table relation cache general and...

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

[GitHub] spark issue #16621: [SPARK-19265][SQL] make table relation cache general and...

2017-01-17 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/16621 Sure. No problem. --- 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 #16621: [SPARK-19265][SQL] make table relation cache general and...

2017-01-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16621 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 #16621: [SPARK-19265][SQL] make table relation cache general and...

2017-01-17 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/16621 can we do it later? We are going to merge `CatalogRelation` implementations and unify the table relation representations soon. --- If your project is set up for it, you can reply to this email

[GitHub] spark issue #16621: [SPARK-19265][SQL] make table relation cache general and...

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

[GitHub] spark issue #16621: [SPARK-19265][SQL] make table relation cache general and...

2017-01-17 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/16621 Could we rename `SimpleCatalogRelation` to `UnresolvedCatalogRelation`? The current name looks very confusing to me. --- If your project is set up for it, you can reply to this email and have

[GitHub] spark issue #16621: [SPARK-19265][SQL] make table relation cache general and...

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

[GitHub] spark issue #16621: [SPARK-19265][SQL] make table relation cache general and...

2017-01-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16621 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 #16621: [SPARK-19265][SQL] make table relation cache general and...

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

[GitHub] spark issue #16621: [SPARK-19265][SQL] make table relation cache general and...

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

[GitHub] spark issue #16621: [SPARK-19265][SQL] make table relation cache general and...

2017-01-17 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/16621 Start the review. Will remove this comment when the review is finished. --- 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 issue #16621: [SPARK-19265][SQL] make table relation cache general and...

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

[GitHub] spark issue #16621: [SPARK-19265][SQL] make table relation cache general and...

2017-01-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/16621 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 #16621: [SPARK-19265][SQL] make table relation cache general and...

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

[GitHub] spark issue #16621: [SPARK-19265][SQL] make table relation cache general and...

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

[GitHub] spark issue #16621: [SPARK-19265][SQL] make table relation cache general and...

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

[GitHub] spark issue #16621: [SPARK-19265][SQL] make table relation cache general and...

2017-01-17 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/16621 cc @yhuai @gatorsmile --- 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